mam/mamweb
13
1
Fork 0
Code Issues Pull requests 12 Projects Wiki Activity Actions

akce na sjednoceni cisel do +420 123 456 789 #53

Merged
zelvuska merged 3 commits from sjednoceni_telefonu into master 2024-06-15 23:33:25 +02:00
Conversation 1 Commits 3 Files changed 1 +22 -1
ticvac commented 2024-06-15 23:00:03 +02:00
Owner
Copy link

Sjednocení telefonů do formátu "+420 123 456 789"
Netestováno!

Sjednocení telefonů do formátu "+420 123 456 789" Netestováno!
ticvac added 1 commit 2024-06-15 23:00:04 +02:00
ticvac changed title from akce na sjednoceni cisel do +420 123 456 78 to akce na sjednoceni cisel do +420 123 456 789 2024-06-15 23:00:47 +02:00
zelvuska reviewed 2024-06-15 23:15:21 +02:00
zelvuska left a comment
Owner
Copy link

Skvělý začátek, ale chybová hláška není pro průměrné orgy (takové ty, co musí mít vše na 3 kliknutí).

Skvělý začátek, ale chybová hláška není pro průměrné orgy (takové ty, co musí mít vše na 3 kliknutí).
personalni/admin.py Outdated
@ -8,0 +21,4 @@
else:
raise ValueError
except:
print("Hmm má divné číslo - " + o.telefon + " - " + str(o.id))
zelvuska commented 2024-06-15 23:07:11 +02:00
Owner
Copy link

Hmm má divné číslo - +42077123881237 (jsem-li v česku); jinak +213893246791425 - 50

Hmm má divné číslo - +42077123881237 (jsem-li v česku); jinak +213893246791425 - 50
zelvuska commented 2024-06-15 23:09:18 +02:00
Owner
Copy link

Představoval bych si spíše

Jonáš Havelka (50) má divné číslo "+42077123881237 (jsem-li v česku); jinak +213893246791425"

Představoval bych si spíše Jonáš Havelka (50) má divné číslo "+42077123881237 (jsem-li v česku); jinak +213893246791425"
ticvac added 1 commit 2024-06-15 23:20:39 +02:00
ticvac added 1 commit 2024-06-15 23:27:46 +02:00
zelvuska approved these changes 2024-06-15 23:33:09 +02:00
zelvuska left a comment
Owner
Copy link

Super :)

Super :)
zelvuska merged commit 40ca76d414 into master 2024-06-15 23:33:25 +02:00
zelvuska deleted branch sjednoceni_telefonu 2024-06-15 23:33:25 +02:00
ledoian commented 2024-06-19 22:51:21 +02:00
Owner
Copy link

No, pořád bych chtěl protestovat proti tomu obecnému exceptu. Mimo jiné to znamená, že když v nešikovnou chvíli zkusím zabít web pomocí Ctrl+C, tak to v Adminu vypíše hlášku o blbosti a web to nezabije.

Celkově je docela dobrý nápad nemít moc dlouhé try bloky a hlídat jen chyby, které umíme řešit (podobně irelevantní je problém s databází, ten taky může nastat kdykoliv…)

Já vím, já si to třeba jednou fixnu, spíš pro příště :-)

No, pořád bych chtěl protestovat proti tomu obecnému exceptu. Mimo jiné to znamená, že když v nešikovnou chvíli zkusím zabít web pomocí Ctrl+C, tak to v Adminu vypíše hlášku o blbosti a web to nezabije. Celkově je docela dobrý nápad nemít moc dlouhé `try` bloky a hlídat jen chyby, které umíme řešit (podobně irelevantní je problém s databází, ten taky může nastat kdykoliv…) Já vím, já si to třeba jednou fixnu, spíš pro příště :-)
👍 1
Sign in to join this conversation.
Reviewers
No reviewers
zelvuska
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
3 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: mam/mamweb#53
No description provided.
Delete branch "sjednoceni_telefonu"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?