-
Notifications
You must be signed in to change notification settings - Fork 281
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 urls #233
base: main
Are you sure you want to change the base?
Conversation
@amyschoen Thank you so much for doing this! ✨ Everything looks correct, I'd just like to wait until I hear back from @brianamarie regarding the possibility of using a custom domain for this repo before approving the changes: #232 (comment). |
You're welcome @parkerbxyz, and it makes sense. github.github.io does look a tad odd in the URL. |
@brianamarie, @parkerbxyz, @BirdyBigBird, I see my WIP PR got approved. Do we have a date for the repo move yet? |
Hi @amyschoen, thank you for following up. At this point, we do not have a date planned for moving and renaming the repository. While it's a move we want to make, the urgency is low and the level of impact is unclear, which makes it harder to prioritize. If there are any reasons or upcoming events that we should be considering, please let us know. 😊 In the meantime, thank you for your contributions and for your patience. 🙏 |
There aren't any timelines on my part @brianamarie. I'll keep this as a WIP for now so it doesn't accidentally get merged in. |
@brianamarie, I've noticed a few approvals lately. Do we have an updated timeline yet for the repo move? |
Hi @amyschoen! I am not sure where these approvals are coming from... especially seeing that one is now "ghost", which usually means that account is no longer active. Unfortunately, I am longer able to look after this initiative as actively. @parkerbxyz, is this something you're shepherding, or would like to shepherd, or is this something we'd consider in the backlog until further notice? |
It's still a draft PR, so no worries about this getting merged in before we're ready. I was just surprised to see a couple approvals recently. |
This is in the backlog, but on my radar. I’m hoping to find time to revisit this soon! 🙂 |
This comment has been minimized.
This comment has been minimized.
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.
demo
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.
demo
This comment has been minimized.
This comment has been minimized.
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.
.
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.
review
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.
GHGH
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.
can be pulled
Fixing hard-coded URLs in preparation for the OSS release.
Closes #232