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

Support specifying short name for team url #11934

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

nanaya
Copy link
Collaborator

@nanaya nanaya commented Feb 25, 2025

It's kinda crap with [...] though since a lot of things don't auto url the closing bracket unless it's percent encoded. I added support for just opening bracket but it's even weirder.

A plus/minus/at/tilde sign or some other prefix would probably work better.

@notbakaneko
Copy link
Collaborator

notbakaneko commented Feb 26, 2025

Well, considering [ ] are supposed to be reserved characters in urls... 👀

@nanaya
Copy link
Collaborator Author

nanaya commented Feb 26, 2025

yeah, [] is only because that's how it's shown on team page.

Also if we're going with supporting short name for url, #11903 wouldn't make sense anymore.

@nanaya nanaya marked this pull request as draft February 26, 2025 06:39
@nanaya
Copy link
Collaborator Author

nanaya commented Feb 26, 2025

Drafting until we finish working out on the details.

  • whether or not allowing duplicate
  • what symbol to use (if going ahead)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants