-
Notifications
You must be signed in to change notification settings - Fork 4.2k
feat: export tagged content library as csv [FC-0049] #34246
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
bradenmacdonald
merged 86 commits into
openedx:master
from
open-craft:rpenido/fal-3611-download-library-tag-spreadsheet
Mar 4, 2024
Merged
Changes from all commits
Commits
Show all changes
86 commits
Select commit
Hold shift + click to select a range
52bc665
feat: export tagged course as csv
rpenido f275908
docs: add comment
rpenido a870dfd
fix: add select_related to ObjectTag query
rpenido 4dd027a
fix: always use objecttag.value
rpenido 5fb03aa
docs: change comment position
rpenido 8f1238e
refactor: rename serializer to a more sane name
rpenido bdedf93
refactor: remove download param
rpenido 04ca072
refactor: create a new view to export objecttags
rpenido a8a6e7e
Merge branch 'master' into rpenido/fal-3610-download-course-tag-sprea…
rpenido b01d6d4
refactor: change api and view structure
rpenido 0871537
Merge branch 'master' into rpenido/fal-3610-download-course-tag-sprea…
rpenido 18a8425
docs: remove old comment
rpenido 38ae353
docs: revert view docstring
rpenido b65a6c8
fix: remove include_children query param
rpenido f5ac3a6
fix: removing .all() call from queryset
rpenido debf254
refactor: method rename
rpenido 9726d6d
fix: filter deleted tags
rpenido c8c12bb
fix: pylint
rpenido 1f11b17
fix: quote string in csv export
rpenido 484c042
test: add querycount
rpenido 6b6ba34
fix: pylint
rpenido 9096454
fix: pylint..
rpenido a688689
fix: pylint
rpenido 30e06d0
test: compare results to hardcoded strings
pomegranited 35a3d2b
test: Adds "deleted" object tags to ensure they are omitted from results
pomegranited 1e13f54
test: adds untagged blocks with children
pomegranited e9335c8
revert: undo removed property
rpenido ab1a69e
style: fix camelCase
rpenido db9116d
refactor: remove xblock from TaggedContent and include_children param
rpenido 9b3dee8
fix: remove UsageKey
rpenido 821e216
fix: removing unused import
rpenido 6207915
refactor: cleaning code
rpenido 35bc860
test: refactors tests so shared data can be re-used
pomegranited 7a28742
refactor: refactoring api, helper and view code
rpenido fabb729
docs: add comment about ObjectTag query
rpenido 548d57c
test: use CourseFactory and BlockFactory
pomegranited a1d41fd
test: fix variable name
rpenido f07b841
fix: delete unwanted file
rpenido 233135a
test: fix query count
rpenido ac98812
test: fix expected value with new tags
rpenido 309ce94
fix: use variables from outer function
rpenido 32cdf93
test: use UserFactory
rpenido 4ed7570
style: removed unused imports
rpenido ee5bc3d
chore: trigger CI
rpenido 4bc8ca7
Merge branch 'master' into rpenido/fal-3610-download-course-tag-sprea…
rpenido 6d4c23a
fix: disable default staff user from module store mixin
rpenido da3fdf9
style: fix case
rpenido 726b7ef
Merge branch 'jill/rpenido/fal-3610-download-course-tag-spreadsheet' …
rpenido fd5a542
docs: adds removed docstring
rpenido f84abd0
feat: export tagged library as csv
rpenido 79a1786
Merge branch 'master' into rpenido/fal-3610-download-course-tag-sprea…
rpenido dfe43be
fix: cleaning merged code
rpenido 5245264
style: run isort
rpenido 06810a2
Merge branch 'rpenido/fal-3610-download-course-tag-spreadsheet' into …
rpenido c82e9cb
refactor: use taxonomy.export_id in header
rpenido f750076
Merge branch 'rpenido/fal-3610-download-course-tag-spreadsheet' into …
rpenido 01b9b5f
refactor: change `object_id` to `context_id`
rpenido f87fc4c
Merge branch 'master' into rpenido/fal-3610-download-course-tag-sprea…
rpenido 779cc98
chore: trigger CI
rpenido f637076
Merge branch 'rpenido/fal-3610-download-course-tag-spreadsheet' into …
rpenido 3cf462f
fix: rebasing
rpenido aa93510
fix: rename variable to fix typing
rpenido e94131f
fix: pylint
rpenido 4a3d092
Merge branch 'master' into rpenido/fal-3610-download-course-tag-sprea…
rpenido 9a1729d
Merge branch 'rpenido/fal-3610-download-course-tag-spreadsheet' into …
rpenido 6518a91
refactor: changing condition checking
rpenido 8ddf875
fix: check invalidkey
rpenido ed547be
fix: better error handling
rpenido df06d9a
docs: reverting unintended change
rpenido a153ec9
refactor: change functions to private
rpenido 610c01a
refactor: cleaning build_object_tree_with_objecttags function
rpenido 4012481
fix: explicit typing
rpenido 4ad87c2
Merge branch 'master' into rpenido/fal-3611-download-library-tag-spre…
rpenido 5caf952
fix: cleaning some types
rpenido df13345
docs: fix comment
rpenido e6a0dce
Merge branch 'master' into rpenido/fal-3611-download-library-tag-spre…
rpenido aa38f08
fix: typo
rpenido 805ed98
Merge branch 'master' into rpenido/fal-3611-download-library-tag-spre…
rpenido 0fd512e
docs: improve docstring
rpenido 11d6de2
fix: remove uwsgi
rpenido 632556c
Merge branch 'master' into rpenido/fal-3611-download-library-tag-spre…
rpenido 83b16cc
Merge branch 'master' into rpenido/fal-3611-download-library-tag-spre…
rpenido 6f02755
fix: update code with upstream changes
rpenido 19c86a6
fix: removing unused import
rpenido 6189a9f
Merge branch 'master' into rpenido/fal-3611-download-library-tag-spre…
rpenido 07e8fa9
Merge branch 'master' into rpenido/fal-3611-download-library-tag-spre…
rpenido File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.