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

refactor: Remove unused user endpoints like activeAuthors #1128

Merged
merged 1 commit into from
Nov 22, 2023

Conversation

kulla
Copy link
Member

@kulla kulla commented Nov 21, 2023

@kulla kulla requested review from elbotho and Entkenntnis November 21, 2023 18:51
@kulla kulla force-pushed the remove-user-endpoints branch from 13dbe13 to 6d11b18 Compare November 21, 2023 19:01
@kulla kulla marked this pull request as draft November 21, 2023 19:07
@kulla
Copy link
Member Author

kulla commented Nov 21, 2023

See https://serlo.slack.com/archives/C062X5BJANP/p1700592998027839 -> I want to check whether there is a bug in my script

@kulla kulla marked this pull request as ready for review November 22, 2023 11:20
@kulla
Copy link
Member Author

kulla commented Nov 22, 2023

@elbotho @Entkenntnis Ready for review. Please double check whether those endpoints are currently used.

@elbotho
Copy link
Member

elbotho commented Nov 22, 2023

can confirm that activeAuthors, activeDonors activeReviewers are not used by the frontend.

@kulla kulla added this pull request to the merge queue Nov 22, 2023
Merged via the queue into main with commit 50a3186 Nov 22, 2023
10 checks passed
@kulla kulla deleted the remove-user-endpoints branch November 22, 2023 18:51
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.

2 participants