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

Fix updateName function #313

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

chriscarrollsmith
Copy link
Contributor

@chriscarrollsmith chriscarrollsmith commented Mar 10, 2024

Added support for Supabase AI
Clarified documentation for local Supabase setup
Updated dependencies
Copy link

vercel bot commented Mar 10, 2024

@chriscarrollsmith is attempting to deploy a commit to the Vercel Solutions Team on Vercel.

A member of the Team first needs to authorize it.

@@ -3,14 +3,15 @@
"version": "0.0.0",
"license": "MIT",
"scripts": {
"dev": "next dev --turbo",
Copy link
Member

Choose a reason for hiding this comment

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

Let's keep this and please share the errors you're seeing! We'd like to fix them. Could you also try on the latest next@canary please?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sure. Here's the error:

Screenshot 2024-03-11 105127

I'm happy to report that it seems fixed in next@canary, though! Should I update the PR to use that, or should I wait until official release of stable 14.2.0?

Regenerated database types to fix Issue 297
Enhanced README OAuth instructions and added a note on controlling display sequence for Stripe products
@chriscarrollsmith
Copy link
Contributor Author

chriscarrollsmith commented Mar 11, 2024

PR updated as follows:

  • Next.js version bump to next@canary
  • Restored --turbo flag
  • Fixed missing types
  • Added mobile-friendly responsive hamburger menu
  • Enhanced README a bit more to add more detail on auth setup and local OAuth testing

@leerob
Copy link
Member

leerob commented Mar 11, 2024

Yeah, go ahead and keep canary 😄

@@ -95,7 +97,7 @@ For example, you can create business models with different pricing tiers, e.g.:
- Price 1: 20 USD per month
- Price 2: 200 USD per year

Optionally, to speed up the setup, we have added a [fixtures file](fixtures/stripe-fixtures.json) to bootstrap test product and pricing data in your Stripe account. The [Stripe CLI](https://stripe.com/docs/stripe-cli#install) `fixtures` command executes a series of API requests defined in this JSON file. Simply run `stripe fixtures fixtures/stripe-fixtures.json`.
Optionally, to speed up the setup, we have added a [fixtures file](fixtures/stripe-fixtures.json) to bootstrap test product and pricing data in your Stripe account. The [Stripe CLI](https://stripe.com/docs/stripe-cli#install) `fixtures` command executes a series of API requests defined in this JSON file. Simply run `stripe fixtures fixtures/stripe-fixtures.json`. (Note the use of a numerical index in the product metadata to control display sequence in the UI).
Copy link
Contributor

Choose a reason for hiding this comment

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

omg thank you for this note! You just helped me figure out how to sequence my products!

@chriscarrollsmith
Copy link
Contributor Author

chriscarrollsmith commented May 10, 2024 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

--turbo flag breaks dependencies updateName doesn't update the name in account page Type error priceData
3 participants