fix: validation of url fields#359
Conversation
|
Looks like you did not link an issue to this PR. If this PR completes a task, consider linking it. |
michalges
left a comment
michalges
left a comment
There was a problem hiding this comment.
można to zrobić trochę prościej, bezpośrenio używając z.string().startsWith()
https://zod.dev/api?id=strings#strings
| .string() | ||
| .trim() | ||
| .url(FORM_ERROR_MESSAGES.INVALID_URL) | ||
| .startsWith("http://", { |
| FORM_ERROR_MESSAGES.INVALID_URL, | ||
| ); | ||
| ).startsWith("http://", { | ||
| message: FORM_ERROR_MESSAGES.INVALID_URL, |
There was a problem hiding this comment.
ten błąd niewiele mówi użytkownikowi, może dodaj nowy w stylu "Adres URL powinien zaczynać się od https://"
There was a problem hiding this comment.
package-lock nie powinien ulec żadnym zmianom w pr'ce gdzie nie instalujesz nic nowego itp. Możesz ręcznie skopiować stary package-lock (z przed twoich zmian) i go z powrotem zacommitować, albo lepiej - wyszukaj sobie jak się "edytuje" commity (interactive rebase) (efekt będzie ten sam, ale ta druga metoda bardziej porządna + można się czegoś zawsze nauczyć)
na przyszłość po prostu nie dodawaj package-lock.json do commitów
2 participants
Fixed url validation