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

Link to github app in .env for slugEnv #1360

Merged
merged 3 commits into from
Nov 21, 2024
Merged

Conversation

tordans
Copy link
Contributor

@tordans tordans commented Nov 21, 2024

In the spirit of #1359, I wonder if the .env file should just have the link to the app on GitHub right there.

This would make the docs more or less obsolete for users who know how to find the app settings once they have their apps page.


Disclaimer: I did not run this code (due to #1285).

Finding the link to the app on GitHub to edit the settings (like adding a `redirect_uri`) is hard. Having the link right where the app is defined means most users will not have to read the docs from Thinkmill#1359.
@tordans tordans requested a review from emmatown as a code owner November 21, 2024 05:47
Copy link

changeset-bot bot commented Nov 21, 2024

🦋 Changeset detected

Latest commit: cf26cef

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 13 packages
Name Type
@keystatic/core Patch
@keystar/docs Patch
@example/astro-content Patch
@example/astro Patch
@example/localization Patch
@example/next-app Patch
@example/next-block-builder Patch
@example/next-pages Patch
@keystatic/remix-test-app Patch
keystatic-docs Patch
@keystatic/templates-astro Patch
@keystatic/templates-nextjs Patch
@keystatic/templates-remix Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@emmatown emmatown merged commit 0d69577 into Thinkmill:main Nov 21, 2024
5 checks passed
@github-actions github-actions bot mentioned this pull request Nov 21, 2024
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.

2 participants