Skip to content

Conversation

@Talha-Rizwan
Copy link
Contributor

@Talha-Rizwan Talha-Rizwan commented Apr 21, 2025

#35352 issue for tracking

Description

Converts the function-based export_ora2_data view to a DRF class-based view
This change maintains all existing functionality while providing a more standardized API structure.

Key changes:

  • Created ExportOra2DataView class inheriting from APIView
  • Maintained existing permission requirements
  • No functional changes, purely architectural improvement for consistency and maintainability.

To Test the api, try POST request on this url

expected response:
{"status": "The ORA data report is being created. To view the status of the report, see Pending Tasks below."}

a celery task will be triggered and a csv file will be generated.

verify via instructor dashbaord

  1. go to this page

  2. click on the button "Generate ORA Data Report"

  3. A csv file will be ready to be downloaded once the celery task is completed

output like this will be generated:
Screenshot 2025-04-21 at 4 23 25 PM

@Talha-Rizwan Talha-Rizwan requested review from a team and feanil as code owners April 21, 2025 11:24
@Talha-Rizwan Talha-Rizwan changed the base branch from export_ora2_data-to-drf to master April 21, 2025 11:25
@Talha-Rizwan
Copy link
Contributor Author

@awais786 PR is ready for review

@Talha-Rizwan Talha-Rizwan force-pushed the export_ora2_data-to-drf branch from c4a3b57 to 26aa6fc Compare April 29, 2025 08:57
@feanil
Copy link
Contributor

feanil commented Jun 2, 2025

@Talha-Rizwan looks like there are some failing tests, can you update the branch to get the build green again?

@Talha-Rizwan Talha-Rizwan force-pushed the export_ora2_data-to-drf branch from 6309107 to 5113c92 Compare June 4, 2025 11:37
@Talha-Rizwan
Copy link
Contributor Author

@feanil I've updated the PR, Please take a look at it whenever convenient.

@feanil
Copy link
Contributor

feanil commented Jun 4, 2025

@awais786 thanks for reviewing, feel free to merge if it looks good to you, you don't need to wait on my review for these if they look good to you.

@awais786 awais786 merged commit b6cec3c into openedx:master Jun 5, 2025
49 checks passed
@edx-pipeline-bot
Copy link
Contributor

2U Release Notice: This PR has been deployed to the edX staging environment in preparation for a release to production.

@edx-pipeline-bot
Copy link
Contributor

2U Release Notice: This PR has been deployed to the edX production environment.

marlonkeating pushed a commit that referenced this pull request Jul 15, 2025
* feat: export ora2 data to DRF
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants