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

Remove usages of EDX_API_KEY #2872

Closed
2 tasks
asadali145 opened this issue Jan 22, 2024 · 3 comments
Closed
2 tasks

Remove usages of EDX_API_KEY #2872

asadali145 opened this issue Jan 22, 2024 · 3 comments

Comments

@asadali145
Copy link
Contributor

asadali145 commented Jan 22, 2024

Open edX is going to remove the EDX_API_KEY. Details can be found at openedx/edx-platform#34039. We have a couple of stale usages that we want to remove. We are using the EDX_API_KEY with the token authentication and we can safely remove EDX_API_KEY as per initial R&D.

Here are the usages:

Acceptance Criteria:

  • Usage of EDX_API_KEY is removed
  • No impact on any of the features or management commands

Similar Issues

mitodl/mitxonline#2102
mitodl/edx-api-client#103

Out of Scope

@arslanashraf7
Copy link
Contributor

@Anas12091101 Could you take a look at this?

@Anas12091101 Anas12091101 self-assigned this Apr 23, 2024
@arslanashraf7
Copy link
Contributor

@mudassir-hafeez Could you take a look at this? I think @Anas12091101 's xPRO integration isn't working so it might be a good idea to take this on at this point.

@mudassir-hafeez
Copy link
Contributor

Closing #2872 as #2982 and mitodl/edx-api-client#104 have been deployed to production.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants