openedx-unsupported / frontend-app-library-authoring

DEPRECATED. To be migrated into frontend-app-course-authoring.
GNU Affero General Public License v3.0
3 stars 25 forks source link

feat: add export library tags menu [FC-0049] #412

Closed rpenido closed 8 months ago

rpenido commented 8 months ago

Description

This PR adds an item in the Settings menu to allow a call of the function that exports the library tags to a CSV file.

image

Supporting Information

Testing instructions

  1. Use https://github.com/open-craft/taxonomy-sample-data/ to create some taxonomies.
  2. Go to the library page in Studio and add some tags to the units in the course using the Manage Tags option image
  3. Checkout THIS branch and reload the library authoring page
  4. Click on the Export Tags menu from the detail page of an existing library
  5. Check if a CSV file is downloaded with the Library structure and the tags applied to its items.

Private-ref:

openedx-webhooks commented 8 months ago

Thanks for the pull request, @rpenido! Please note that it may take us up to several weeks or months to complete a review and merge your PR.

Feel free to add as much of the following information to the ticket as you can:

All technical communication about the code itself will be done via the GitHub pull request interface. As a reminder, our process documentation is here.

Please let us know once your PR is ready for our review and all tests are green.

codecov[bot] commented 8 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (de8582a) 53.14% compared to head (e4d7c82) 53.14%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #412 +/- ## ======================================= Coverage 53.14% 53.14% ======================================= Files 80 80 Lines 2021 2021 Branches 368 368 ======================================= Hits 1074 1074 Misses 914 914 Partials 33 33 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

itsjeyd commented 8 months ago

@rpenido @pomegranited It looks like these changes are ready for upstream review. I'm changing status on the Contributions board accordingly, but let me know if you have any objections.

@navinkarkera Could you give this a maintainer review?

rpenido commented 8 months ago

Hi @navinkarkera! Thank you for the review. The edx-platform PR got merged!

openedx-webhooks commented 8 months ago

@rpenido 🎉 Your pull request was merged! Please take a moment to answer a two question survey so we can improve your experience in the future.