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

Adding codeorg-api.com and xeno-canto.org #44520

Merged
merged 1 commit into from
Jan 25, 2022
Merged

Conversation

KylieModen
Copy link
Contributor

@KylieModen KylieModen commented Jan 25, 2022

https://xeno-canto.org/explore/api is a bird sounds API requested by a student https://codeorg.zendesk.com/agent/tickets/370728

https://docs.codeorg-api.com/ is a student created API to connect two or more clients through a server for game development https://codeorg.zendesk.com/agent/tickets/369165

https://xeno-canto.org/explore/api is a bird sounds API requested by a student https://codeorg.zendesk.com/agent/tickets/370728

https://docs.codeorg-api.com/ is a student created API to connect two or more clients through a server for game development https://codeorg.zendesk.com/agent/tickets/369165
@KylieModen KylieModen requested a review from a team January 25, 2022 17:35
@@ -63,6 +63,7 @@ class XhrProxyController < ApplicationController
api.zippopotam.us
bible-api.com
code.org
codeorg-api.com
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we should add a comment (maybe with a link to the documentation) to make it clear that this API isn't owned/maintained by code.org. ruby doesn't support comments in a %w() block like this, so the comment will have to go at the top (where the other 'code.org' comment is), unfortunately

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great callout Maddie - I'll add that comment

@KylieModen KylieModen merged commit 10dda72 into staging Jan 25, 2022
@KylieModen KylieModen deleted the KylieModen-patch-3 branch January 25, 2022 19:55
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.

None yet

2 participants