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.

Sign up for GitHub

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

Jump to bottom

[DEPR]: Remove EdxRestApiClient from edx-rest-api-client library #218

Open
2 tasks
dianakhuang opened this issue Oct 19, 2023 · 2 comments
Open
2 tasks

[DEPR]: Remove EdxRestApiClient from edx-rest-api-client library #218

dianakhuang opened this issue Oct 19, 2023 · 2 comments
Assignees
@Yagnesh1998
Labels
depr Proposal for deprecation & removal per OEP-21

Comments

@dianakhuang
Copy link

As part of the work to complete #189

We would like to:

  • Move EdxRestApiClient into a location near where it is used in edx-platform and rename it to DeprecatedRestApiClient
  • Remove EdxRestApiClient from https://github.com/openedx/edx-rest-api-client with proper release notes and versioning indicating that it is a breaking release.
@github-actions github-actions bot added the depr Proposal for deprecation & removal per OEP-21 label Oct 19, 2023
@dianakhuang dianakhuang changed the title [DEPR]: Remove EdxRestApiClient [DEPR]: Remove EdxRestApiClient from edx-rest-api-client library Oct 19, 2023
@Yagnesh1998
Copy link

assign me

@Yagnesh1998 Yagnesh1998 mentioned this issue Oct 26, 2023
Closed
@dianakhuang dianakhuang moved this from Proposed to Accepted in DEPR: Deprecation & Removal Nov 2, 2023
@dianakhuang dianakhuang moved this from Accepted to Removing in DEPR: Deprecation & Removal Nov 2, 2023
@Yagnesh1998 Yagnesh1998 mentioned this issue Nov 6, 2023
Merged
@robrap
Copy link

robrap commented Aug 26, 2024

@dianakhuang:

  1. I don't think @Yagnesh1998 is working on this any longer, so we should probably unassign.
  2. It is possible that feat: DEPR EdxRestApiClient edx-rest-api-client#303 is the only remaining work, and was really clost to landing, so we could possibly do this in a DEPR meeting.

@robrap robrap mentioned this issue Aug 26, 2024
Closed
9 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Yagnesh1998 Yagnesh1998

Labels
depr Proposal for deprecation & removal per OEP-21
Projects
DEPR: Deprecation & Removal
Status: Removing
Milestone
No milestone
Development

No branches or pull requests
3 participants
@dianakhuang @robrap @Yagnesh1998