Expand app quick start guides #1106
Conversation
azigler
added
Work in Progress
|
Preview environment failed. |
1 similar comment
|
Preview environment failed. |
|
Newest code from azigler has been published to preview environment for Git SHA 80371ba |
|
/update-branch |
|
Newest code from mattermod has been published to preview environment for Git SHA 01ff486 |
|
Newest code from azigler has been published to preview environment for Git SHA 1c60676 |
- tidy up language
|
Newest code from azigler has been published to preview environment for Git SHA f18c650 |
- clean up language in other quick start guides
|
/update-branch |
site/content/integrate/apps/quickstart/quick-start-serverless/_index.md
Outdated
Show resolved
Hide resolved
site/content/integrate/apps/quickstart/quick-start-ts/_index.md
Outdated
Show resolved
Hide resolved
site/content/integrate/apps/quickstart/quick-start-ts/_index.md
Outdated
Show resolved
Hide resolved
|
Newest code from cwarnermm has been published to preview environment for Git SHA 1acf0cf |
|
Newest code from levb has been published to preview environment for Git SHA 18451aa |
|
@azigler - The dev docs site has seen a lot of changes recently. Given the age of this PR, would you be open to reviewing the changes proposed in this PR against content available in production to confirm whether we want to move ahead with this PR? |
|
@cwarnermm Thanks for checking about this one. From what I can tell, there are just tiny conflicts on two of the quickstart pages in this PR currently, and both can just be resolved in favor of this PR because it's a full revamp of those pages. Our current developer tutorials for apps are very spare, so I think getting this out soon (especially at the dawn of Hacktoberfest) is best, and then we can continue to iterate -- it's great to see we have a lot of ideas about improving these docs! Despite the age, I don't think any of the content is expired, it seems to be working on my end. I see Alan has feedback about file permissions when running Docker, but that's a separate Docker issue outside of this guide that I can help address with him. Let me know how you'd like to proceed! |
|
Sounds great, @azigler! Would you be open to resolving the outstanding feedback and merge conflicts on this PR? Once resolved, we can merge this great update into production. |
Co-authored-by: Alan Lew <[email protected]>
Co-authored-by: Alan Lew <[email protected]>
|
/update-branch |
|
Newest code from azigler has been published to preview environment for Git SHA 9310637 |
Do you hear that? The quiet stillness? That's the sound of 214 individually resolved conversations! This PR has been an incredible effort by everyone -- @neflyte @cwarnermm @mickmister these guides wouldn't exist without y'all's sagely perspectives! 🧙 I resolved Alan's comments, incorporated the changes, and fixed the conflicts. I also merged master into this branch to update it. It failed to build at first, but then I fixed the link to @jordanafung's new contributor pages and now it's good! I'd love to see this get merged to bolster Hacktoberfest app integrators and put resources in their hands ASAP. This is a night-and-day difference over what's currently there, and once it's live we'll continue to iterate because it's clear we all care very much about making these guides successful -- and I love it! ✨ @cwarnermm please let me know if there are any obstacles to merge at this time. |
|
Newest code from cwarnermm has been published to preview environment for Git SHA 0907ad5 |
cwarnermm
added
4: Reviews Complete
|
Newest code from cwarnermm has been published to preview environment for Git SHA 0907ad5 |
Fixes:
Adds:
Blocked guides:
ngrok, will require more thought./hello-webhooks infoendpoint does nothing when sent to, app does not report messages to channel.