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

Close #476 #480

Merged
merged 1 commit into from
May 6, 2024
Merged

Close #476 #480

merged 1 commit into from
May 6, 2024

Conversation

bobronaud
Copy link
Contributor

пофиксил возможность создавать сниппеты с пустым именем
ишьюс - #476

@fey
Copy link
Contributor

fey commented May 6, 2024

А валидацию поправить? =)

@bobronaud
Copy link
Contributor Author

@fey не понимаю, что поправить?)

@fey
Copy link
Contributor

fey commented May 6, 2024

@bobronaud сейчас на фронте можно ввести пробелы, они обрезаются, да. Но по правильному тут стоит поправить валидацию, чтобы в имени сниппета нельзя было указать только пробелы.
т.е. если бы у нас была изначально корректная валидация, то мы бы не могли создавать сниппеты только с пробельными именами.

@bobronaud bobronaud force-pushed the fix-empty-snippetName branch from 25db4f9 to cdd5200 Compare May 6, 2024 13:06
@bobronaud
Copy link
Contributor Author

@fey добавил валидацию

@fey fey merged commit e3db524 into hexlet-rus:main May 6, 2024
2 checks passed
@bobronaud bobronaud deleted the fix-empty-snippetName branch June 3, 2024 10:10
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