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

Include host in authenticated API responses for events #157

Merged
merged 1 commit into from
Sep 10, 2024

Conversation

lavaturtle
Copy link

We recently added host information to authenticated REST API responses about events. This updates the documentation to match, by including the host block in the example responses from the List, Show, and Create endpoints.

image
image

@lavaturtle lavaturtle added the merge-candidate This PR is ready to be merged, once it is reviewed and tested label Sep 10, 2024
Copy link

@anero anero left a comment

Choose a reason for hiding this comment

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

👍

@lavaturtle lavaturtle added the on-hold-do-not-merge This PR should not be merged now. External dependencies must be resolved first. label Sep 10, 2024
@lavaturtle
Copy link
Author

Will merge and deploy once we've deployed the code change.

@lavaturtle lavaturtle removed the on-hold-do-not-merge This PR should not be merged now. External dependencies must be resolved first. label Sep 10, 2024
@lavaturtle lavaturtle merged commit 37b4104 into mf-cs-documentation Sep 10, 2024
2 checks passed
@lavaturtle lavaturtle deleted the event-host branch September 10, 2024 16:23
lavaturtle added a commit that referenced this pull request Sep 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
merge-candidate This PR is ready to be merged, once it is reviewed and tested
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants