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

Decide on special characters to accept/reject #221

Open
dreeves opened this issue Jun 25, 2018 · 1 comment
Open

Decide on special characters to accept/reject #221

dreeves opened this issue Jun 25, 2018 · 1 comment
Labels
ADO Consensus needed on what to Actually Do (or "much ado about ∅"), AKA question MEN Mendoza = need to resolve before accepting more beta users UVI User-Visible Improvement
Projects

Comments

@dreeves
Copy link
Contributor

dreeves commented Jun 25, 2018

My Latest Arguments Against Slashes In Particular

  1. We may want slashes to have special meaning so we shouldn't allow them willy-nilly in the meantime and make a (bigger) backwards compatibility mess.
  2. Bots and humans try dropping stuff after slashes, like going up a directory. This would be a feature in the case of @chrisbutler's folders idea but is bad in the status quo.
  3. A related problem is that it's not obvious that URLs like alice.commits.to/reply_to_bob/by/5pm and alice.commits.to/reply_to_bob/by/4pm are treated as entirely distinct. Change those to alice.commits.to/reply_to_bob_by_5pm and alice.commits.to/reply_to_bob_by_4pm and it's more clear that the system won't try to do any magic to treat those as referring to the same underlying promise.
@dreeves dreeves added MEN Mendoza = need to resolve before accepting more beta users ADO Consensus needed on what to Actually Do (or "much ado about ∅"), AKA question UVI User-Visible Improvement labels Jun 25, 2018
@dreeves
Copy link
Contributor Author

dreeves commented Jun 26, 2018

In fact, I believe that if we did want to accept slashes we'd need to implement a solution like slash blindness: https://github.com/commitsto/commits.to/wiki#option-2-slash-blindness

@dreeves dreeves added this to TRIAGE in Commits.to Jul 13, 2018
@dreeves dreeves moved this from TRIAGE to TODO in Commits.to Jul 13, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ADO Consensus needed on what to Actually Do (or "much ado about ∅"), AKA question MEN Mendoza = need to resolve before accepting more beta users UVI User-Visible Improvement
Projects
Commits.to
  
TODO
Development

No branches or pull requests

1 participant