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(showcase): update wrong url #303

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

jsparkdev
Copy link

Replace Github repository url with website url.

@machour
Copy link
Contributor

machour commented Oct 29, 2024

Not sure about this change : The repo links to the website, but the website doesn't seem to link back to the repo.
I'd keep it like it currently is.

@jsparkdev
Copy link
Author

jsparkdev commented Oct 29, 2024

@machour

Thanks.

Is this an intentional link to a Github repository?

If so, I will close this PR.

@machour
Copy link
Contributor

machour commented Oct 29, 2024

@jsparkdev Adding this entry was suggested here: remix-run/remix#7503 (comment)

At the time, we did distinguish between website & repo, but I guess we got rid of repo since a lot of sites have closed sources and it was empty most of the time.

let's defer to @brookslybrand for the final call 🙌🏼

@jsparkdev
Copy link
Author

@machour I didn't realize this had been discussed so much before, thanks for the info! I will keep this PR for now.

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