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

chore(console): Enable custom domains and designer after paid plans implementation #2486

Conversation

Cosmin-Parvulescu
Copy link
Contributor

@Cosmin-Parvulescu Cosmin-Parvulescu commented Jul 13, 2023

Description

  • Add proper route names to early access features
  • Add nav links to the nicely named routes
  • Adds client-side differential rendering for features not available in current app plan
  • Adds server side client checks for features not available in current app plan
  • Adds helper functions for dealing with plan differences
  • Provisions a toast system that might be useful in the future

Related Issues

Testing

  • Clicking in nav menu
  • Trying to run action for PRO plan from a FREE plan app
  • Checking visual changes applied in a PRO app from the Designer screen
  • Downgrading the app to FREE
  • Noticing the custom design downgrade

Testing in dev

  • Create free app
  • Check out default app theme
  • Check out default e-mail (hostname & content)
  • Upgrade to PRO
  • Configure app theme, e-mail and custom domains
  • Check out that custom hostname works
  • Check out themed passport
  • Check out themed e-mail
  • Check out delivery hostname on OTP e-mail
  • Downgrade to FREE
  • Notice that custom hostname no longer works
  • Check out default themed passport
  • Check out default themed e-mail
  • Check out default hostname on OTP e-mail

Checklist

  • I have read the CONTRIBUTING guidelines
  • I have tested my code (manually and/or automated if applicable)
  • I have updated the documentation (if necessary)

@Cosmin-Parvulescu Cosmin-Parvulescu added the enhancement Indicates new feature requests label Jul 13, 2023
@Cosmin-Parvulescu Cosmin-Parvulescu self-assigned this Jul 13, 2023
@Cosmin-Parvulescu Cosmin-Parvulescu marked this pull request as ready for review July 13, 2023 14:03
@Cosmin-Parvulescu Cosmin-Parvulescu marked this pull request as draft July 13, 2023 14:56
@Cosmin-Parvulescu Cosmin-Parvulescu marked this pull request as ready for review July 13, 2023 15:23
@Cosmin-Parvulescu Cosmin-Parvulescu force-pushed the chore/console/enable-custom-domains-and-designer-after-paid-plans-implementation branch from 501c621 to 2ab1268 Compare July 13, 2023 19:18
import { ToastType } from '@proofzero/design-system/src/atoms/toast'
import { commitFlashSession } from '~/utilities/session.server'

const isPlanGuarded = async (
Copy link
Contributor

Choose a reason for hiding this comment

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

This doesn't need to be async. Likewise, no need to await it in the function below.

@@ -984,7 +984,7 @@ export default () => {
}),
},
{
Copy link
Contributor

Choose a reason for hiding this comment

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

This button disappears when entering the details (name and email), but is still clickable.
image

Copy link
Contributor

Choose a reason for hiding this comment

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

May be a local setup thing on my side as others can't reproduce. Proceeding.

@@ -984,7 +984,7 @@ export default () => {
}),
},
{
Copy link
Contributor

Choose a reason for hiding this comment

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

May be a local setup thing on my side as others can't reproduce. Proceeding.

@betimshahini betimshahini merged commit 022cbf8 into main Jul 13, 2023
14 of 16 checks passed
@betimshahini betimshahini deleted the chore/console/enable-custom-domains-and-designer-after-paid-plans-implementation branch July 13, 2023 20:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Indicates new feature requests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

chore(console): Enable custom domains and designer after paid plans implementation
2 participants