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 email addresses from API output #109

Open
patrickakk opened this issue Sep 3, 2024 · 5 comments
Open

Remove email addresses from API output #109

patrickakk opened this issue Sep 3, 2024 · 5 comments
Assignees
Labels
done Development and review process finished. Ready for release - Status of issue frontend Front End - Part of application
Milestone

Comments

@patrickakk
Copy link
Contributor

(This was discussed last year, for some reason I couldn't find the issue, so here is a new one)

The courses output from the API contains also email addresses.

For privacy reasons, etc. I would recommend to remove them from the API output and only show on the web page at the course detail page.

@patrickakk patrickakk added frontend Front End - Part of application todo Ready to start development - Status of issue labels Sep 3, 2024
@patrickakk patrickakk added this to the 2024 milestone Sep 3, 2024
@vronk
Copy link

vronk commented Sep 4, 2024

Agree to remove email from API result.

@vronk
Copy link

vronk commented Sep 4, 2024

If it is a matter of under an hour, let's do it now, otherwise postpone.

@patrickakk patrickakk self-assigned this Sep 4, 2024
@patrickakk patrickakk modified the milestones: 2024, Oktober 2024 Sep 4, 2024
@patrickakk patrickakk modified the milestones: 2024-10, 2024-11 Oct 30, 2024
@patrickakk patrickakk added Blocked Blocked by other action – Status of issue and removed todo Ready to start development - Status of issue labels Nov 3, 2024
@patrickakk patrickakk assigned patrickakk and unassigned patrickakk Nov 3, 2024
@patrickakk
Copy link
Contributor Author

@patrickakk patrickakk assigned PixlTracer and IvdL22 and unassigned patrickakk Nov 6, 2024
@patrickakk patrickakk added InReview Request from dev to test and give feedback on the implementation of the issue - Status of issue and removed Blocked Blocked by other action – Status of issue labels Nov 6, 2024
@patrickakk
Copy link
Contributor Author

@PixlTracer @IvdL22

Can you review this? Urls can be found above.

@PixlTracer
Copy link

hi @patrickakk
I've checked the following URLs:

I'll set the label to done.
thank you!

@PixlTracer PixlTracer added done Development and review process finished. Ready for release - Status of issue and removed InReview Request from dev to test and give feedback on the implementation of the issue - Status of issue labels Nov 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
done Development and review process finished. Ready for release - Status of issue frontend Front End - Part of application
Projects
None yet
Development

No branches or pull requests

4 participants