-
Notifications
You must be signed in to change notification settings - Fork 242
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
Нечувствительный к регистру эндпоинт юзеров #1035
base: master
Are you sure you want to change the base?
Conversation
#886 залинкую |
Не уверен, что это ничего не сломает. В самой моделе нет валидации и доп логики на регистронезависимость, и есть сценарии, где можно передать любой слаг, а модель его таким в базу и положит. Даже у меня в профиле слаг имеет ловер и аперкейс, и юзер спокойно может создать такой же ник но с ловеркейсами, что в целом не проблема, но после твоего фикса это сломается. |
@dolshevsk, да, тогда, возможно, стоит глянуть, что там с уникальностью ников с учётом регистронезависимости, и может докинуть ещё миграцию и фикс, чтобы быть уверенными в уникальности |
Можно, но зачем? Какую проблему мы решаем?) Что плохого в том, что регистр учитывается? |
@dolshevsk, я просто взял тикет и фигачу, чтобы немножко мозг развеять и залезть обратно в Джангу после нескольких лет :) |
У нас основная проблема не в том, что не кому закрывать тикеты, а в том, насколько нужны/не нужны/что с дизайном/етс |
@kix, привет. Я решил пройтись по старым задачам. Исправление может быть полезным, но, как заметили, важно проверять это и при создании профиля, чтобы не было конфликтов имен. Я не знаю, актуально ли это сейчас, можно попробовать доделать и проверить, или просто забить на это как на минорный баг. Выбирай. |
С одной стороны, я согласен, удобнее без учёта регистра (я сам при логине свой ник когда-то указал с большой буквы и теперь каждый раз путаюсь, когда ввожу его с маленькой и система не находит пользователя). |
УРЛы профилей будут открываться независимо от регистра:
https://vas3k.club/user/vas3k/
откроет тот же профиль, что иhttps://vas3k.club/user/Vas3k/
slugify
регистронезависим, так что вроде ничего не должно сломаться.Fixes #886