-
Notifications
You must be signed in to change notification settings - Fork 2k
📚 Add vercel to deployment page #6117
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
base: develop
Are you sure you want to change the base?
Conversation
Add Vercel to the examples
WIP (you need to have an own domain, need to discuss with Penpot whether a domain can be provided for plugins or a separate extension at https://penpot.app/.... since you cant use Vercel domain to deploy
Add image links for Vercel
Small explanation to import project in Vercel
Add website link to Title of chapter
Forgot to add wesite link for Surge
Add Vercel logo
Add Netlify and Cloudflare logo
Update logos changed hight to 32px
Insert logos Vercel, Netlify and Cloudflare
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.
Hey.
Thank you very much for your contribution. We really appreciate your time and effort. Before we can proceed with your PR, there are a few important points we'd like to address:
- Please take a moment to review our contributing page, as your PR doesn't currently follow the guidelines.
- The guide appears to be incomplete—it doesn't cover potential CORS issues or the full deployment process. We'd recommend reviewing our existing guides and aligning your content accordingly.
- Please avoid including unnecessary images and ensure that the current format is respected. Specifically, the use of logos should be removed.
- We also ask that you pay closer attention to typos, capitalization, and overall formatting to maintain the quality and consistency of our documentation.
- Lastly, the section numbering requires correction—there are currently two sections labeled 3.3 and no 3.2. Your content should be placed after the "Surge" section, making it the new section 3.5.
Once these adjustments are made, we would be happy to review your contribution again. Thank you once again for your initiative and collaboration.
Hey Done
Done
Did my best, myself, I cannot see how the layout will look definitively. This makes it impossible to check certain things.
Done, Vercel does not belong at the very bottom of the list so have moved it up a bit.
|
Related Ticket
Summary
Add documentation for https://vercel.com to deploy plugin
Steps to reproduce
Checklist
develop
by default.CHANGES.md
file, referencing the related GitHub issue, if applicable.