-
Notifications
You must be signed in to change notification settings - Fork 5
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
Remove refs to old plans #675
Conversation
[Urgent Release] Bug fix for new app creation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@commoddity Confirming that new applications and upgrades still work with this change?
Applied suggestions but unclear what the example links should point to. Co-authored-by: Daniel Olshansky <[email protected]>
Thanks for the review @Olshansk. Just a note for you and @fredteumer that we still need to determine what to do about the failing Vercel .env action: See comment here: #673 (comment) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See #675 (comment)
@Olshansk made requested changes. Ready to merge when you approve. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
approved. let's see if this resolves the CI issue with our friend Vercel
Overview
closes #673
This PR removes all references to old plan types, as well as updates the README.md with more detail on the deployment process.