Skip to content

Update petition image keys & URLs - #196

Merged
lavaturtle merged 3 commits into
mf-cs-documentationfrom
update-image-urls
Aug 12, 2026
Merged

Update petition image keys & URLs#196
lavaturtle merged 3 commits into
mf-cs-documentationfrom
update-image-urls

Conversation

@lavaturtle

@lavaturtle lavaturtle commented Aug 7, 2026

Copy link
Copy Markdown

This makes several updates to example JSON response data:

  • Updates all example image URLs to look like our ActiveStorage image URLs
  • For petitions, updates the image keys to match the new keys (leaving out keys that may still exist but are now deprecated)
  • For the authenticated API, updates the "event show" and "event index" example responses to reflect the keys that are actually included now. This includes image keys but also other keys that had become out of date.

We should wait to merge & deploy this until the corresponding code changes have been deployed to production.

@lavaturtle
lavaturtle marked this pull request as ready for review August 11, 2026 14:28
@lavaturtle lavaturtle added merge-candidate This PR is ready to be merged, once it is reviewed and tested on-hold-do-not-merge This PR should not be merged now. External dependencies must be resolved first. labels Aug 11, 2026
@lavaturtle
lavaturtle requested a review from ehimen-io August 12, 2026 19:34
@lavaturtle lavaturtle removed the on-hold-do-not-merge This PR should not be merged now. External dependencies must be resolved first. label Aug 12, 2026

@ehimen-io ehimen-io left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM!

@lavaturtle
lavaturtle merged commit c90176a into mf-cs-documentation Aug 12, 2026
2 checks passed
@lavaturtle
lavaturtle deleted the update-image-urls branch August 12, 2026 20:07
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