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

🐛 Fixed button URL suggestions not loading for contributors, editors and authors #20416

Merged
merged 1 commit into from
Jun 20, 2024

Conversation

sagzy
Copy link
Contributor

@sagzy sagzy commented Jun 19, 2024

ref https://linear.app/tryghost/issue/SLO-127

  • problem: when using a card with a button (Button, Email CTA, Header, Product), the Button URL suggestions fail to load for Contributors, Authors, and Editors
  • cause: Contributors, Authors and Editors don’t have permission to fetch offers, and this causes the entire list of button url suggestions to break
  • solution: if offers fail to fetch for any reason, the rest of the url suggestions for cards with a button is now still populated (i.e. offers URLs are ignored)

@github-actions github-actions bot added the affects:admin Anything relating to Ghost Admin label Jun 19, 2024
@sagzy sagzy changed the title Fixed offers fetch breaking all button url suggestions 🐛 Fixed button URL suggestions not loading for contributors, editors and authors Jun 19, 2024
@sagzy sagzy force-pushed the fix/SLO-127 branch 2 times, most recently from 3fb8b02 to 4bd609d Compare June 20, 2024 12:11
ref https://linear.app/tryghost/issue/SLO-127
- if offers fail to fetch for some reason (e.g. permission), the rest of the url suggestions for cards with a button (Button, Product, Email CTA) should still work
- as an example use case, contributors do not have access to browse offers. They currently don't see any button url suggestions, not even the default links to the homepage or free signup
@sagzy sagzy merged commit 1c972c7 into main Jun 20, 2024
19 checks passed
@sagzy sagzy deleted the fix/SLO-127 branch June 20, 2024 12:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
affects:admin Anything relating to Ghost Admin
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant