Skip to content
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

feat: get_students_who_may_enroll update api to DRF ( 8th ) #35174

Merged
merged 7 commits into from
Aug 29, 2024

Conversation

awais786
Copy link
Contributor

@awais786 awais786 commented Jul 25, 2024

issue

Upgrading plain api to DRF compatible.

Post man verification

  1. Use this URL as post request.

expected result
It will generate cvs inside instructor dashboard -> datadownload.

verify via dashboard

  1. Go to this page
  2. Click the button Download a CSV of learners who can enroll.

expected result

It will generate cvs inside instructor dashboard -> datadownload.

Screenshot 2024-08-19 at 8 44 28 PM

@awais786 awais786 changed the title Get students who may enroll feat: get_students_who_may_enroll update api to DRF Jul 31, 2024
@awais786 awais786 changed the title feat: get_students_who_may_enroll update api to DRF feat: get_students_who_may_enroll update api to DRF ( 8th ) Aug 19, 2024
@awais786 awais786 requested a review from feanil August 20, 2024 14:30
@awais786
Copy link
Contributor Author

@feanil Please review

@awais786 awais786 marked this pull request as ready for review August 29, 2024 12:48
@awais786 awais786 merged commit 1a431b6 into master Aug 29, 2024
49 checks passed
@awais786 awais786 deleted the get_students_who_may_enroll branch August 29, 2024 13:37
@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.

1 similar comment
@edx-pipeline-bot
Copy link
Contributor

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

@edx-pipeline-bot
Copy link
Contributor

2U Release Notice: This PR has been rolled back from the edX production environment.

@edx-pipeline-bot
Copy link
Contributor

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

1 similar comment
@edx-pipeline-bot
Copy link
Contributor

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

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.

3 participants