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

Ghost shows unconfigured paid tier with filter="visibility:public+type=paid" #15759

Closed
1 task done
fueko opened this issue Nov 2, 2022 · 7 comments
Closed
1 task done
Labels
affects:api Affects the Ghost API bug [triage] something behaving unexpectedly help wanted [triage] Ideal issues for contributors to help with stale [triage] Issues that were closed to to lack of traction

Comments

@fueko
Copy link
Contributor

fueko commented Nov 2, 2022

Issue Summary

Ghost shows an unconfigured paid tier $5/$50 with filter="visibility:public+type:paid" and an unconnected Stripe also this tier cannot be removed in the Portal options.

Zrzut ekranu 2022-11-2 o 20 24 19

Steps to Reproduce

Test the setting for {{#get}} helper with filter="visibility:public+type:paid" without configuring Stripe and it will return a paid tier of $5/$50.

Ghost Version

5.22.4

Node.js Version

v16.13.0

How did you install Ghost?

Tested on localhost and server

Database type

SQLite3

Browser & OS version

No response

Relevant log / error output

No response

Code of Conduct

  • I agree to be friendly and polite to people in this repository
@github-actions github-actions bot added the needs:triage [triage] this needs to be triaged by the Ghost team label Nov 2, 2022
@ErisDS
Copy link
Member

ErisDS commented Nov 15, 2022

Hey @fueko, so sorry for taking such a long time to get back to you on this one!

We agree the API behaviour is a little odd in this case, but from the perspective of creating themes, the best approach is to wrap the get helper in {{#if @site.paid_members_enabled}}{{/if}} (docs here). Using the feature flags for members you have full control over the behaviour of your themes based on the different settings.

In that regard - if there are any more feature flags that would be useful to help make themes more robust against the ever-expanding membership & newsletter settings, let us know!

I'll leave this open, as it would still probably make sense to change the tiers API to return nothing if paid memberships are not enabled.

@ErisDS ErisDS added bug [triage] something behaving unexpectedly affects:api Affects the Ghost API labels Nov 15, 2022
@github-actions github-actions bot removed the needs:triage [triage] this needs to be triaged by the Ghost team label Nov 15, 2022
@ErisDS ErisDS added help wanted [triage] Ideal issues for contributors to help with needs:triage [triage] this needs to be triaged by the Ghost team labels Nov 15, 2022
@github-actions github-actions bot removed the needs:triage [triage] this needs to be triaged by the Ghost team label Nov 15, 2022
@fueko
Copy link
Contributor Author

fueko commented Nov 16, 2022

Thanks very much for the clarifications, @ErisDS. I completely forgot about this helper, thanks!

@snoseeds
Copy link

snoseeds commented Dec 7, 2022

Hi @ErisDS, kindly assign this issue to me

@snoseeds
Copy link

snoseeds commented Dec 9, 2022

Hi @ErisDS, gentle reminder on this

@ErisDS
Copy link
Member

ErisDS commented Dec 16, 2022

@snoseeds A PR would be very welcome. We only assign issues to regular contributors.

@farhan787
Copy link

@ErisDS, I've opened a PR to fix this, need first time contributor approval to run the PR workflows.

@github-actions
Copy link
Contributor

Our bot has automatically marked this issue as stale because there has not been any activity here in some time.

The issue will be closed soon if there are no further updates, however we ask that you do not post comments to keep the issue open if you are not actively working on a PR.

We keep the issue list minimal so we can keep focus on the most pressing issues. Closed issues can always be reopened if a new contributor is found. Thank you for understanding 🙂

@github-actions github-actions bot added the stale [triage] Issues that were closed to to lack of traction label Apr 23, 2023
@github-actions github-actions bot closed this as not planned Won't fix, can't repro, duplicate, stale Apr 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
affects:api Affects the Ghost API bug [triage] something behaving unexpectedly help wanted [triage] Ideal issues for contributors to help with stale [triage] Issues that were closed to to lack of traction
Projects
None yet
Development

No branches or pull requests

4 participants