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

Wrong/Deprecated bulk update history URL being used #349

Open
arslanashraf7 opened this issue Sep 4, 2023 · 0 comments
Open

Wrong/Deprecated bulk update history URL being used #349

arslanashraf7 opened this issue Sep 4, 2023 · 0 comments

Comments

@arslanashraf7
Copy link

While looking into an issue related to grade book MFE, We noticed that one of the URLs was throwing 404. That's the /bulk-update-history. Our assumption is that the bulk update URL might also be wrong.

Steps to reproduce

  • Enable grades.bulk_management Waffle flag in Django Admin (LMS)
  • Run the grade book and monitor the network tab.
  • Notice that It would hit the URL(api/grades/v1/gradebook/course-v1:MITx+Template.Mac+07_25_2023/bulk-updatehistory/) to fetch the bulk update history that ends up in 404 because this URL doesn't seem to exist anywhere. (Here is the code reference from the gradebook MFE).

Possible Solution:

We think that the gradebook MFE should probably use api/bulk_grades/course/course-v1:Arbisoft+ARB_1+1/history/ instead of api/grades/v1/gradebook/course-v1:MITx+Template.Mac+07_25_2023/bulk-updatehistory/.

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

No branches or pull requests

1 participant