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

7.create user and user mangment #3

Open
wants to merge 2 commits into
base: dev
Choose a base branch
from
Open

7.create user and user mangment #3

wants to merge 2 commits into from

Conversation

romcio
Copy link
Collaborator

@romcio romcio commented Nov 15, 2024

Zarządzanie użytkownikiem w oparciu o bazę danych SQLite
jira task PYDZR6MON-7

Usuwa użytkownika o podanym ID.
Zwraca True jeśli się udało, False jeśli wystąpił błąd.
"""
try:
Copy link
Collaborator

@ilsalunte ilsalunte Nov 17, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Czy pod blokiem try powinno być tyle działań?

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Zależy :) Tutaj dwie linijki, więc raczej ok. Zależy czego się spodziewamy, bo Exception jest dość zagadkowe. Im lepiej opisane błędy tym lepiej.

@ilsalunte
Copy link
Collaborator

Chciałam podpytać czy chcemy, żeby istaniała możliwość dodawania różnych użytkowników z tym samym mailem? Póki co istnieje :) Teraz to pewnie bez znaczenia, kiedy będzie jeden user, ale zastanawiam się jak dalej. Chyba, że do Django w ogóle będziemy zmieniali dodawanie użytkowników. Nie wiem jak to tam działa.

@dylodylo
Copy link
Collaborator

dylodylo commented Nov 19, 2024

Chciałam podpytać czy chcemy, żeby istaniała możliwość dodawania różnych użytkowników z tym samym mailem? Póki co istnieje :) Teraz to pewnie bez znaczenia, kiedy będzie jeden user, ale zastanawiam się jak dalej. Chyba, że do Django w ogóle będziemy zmieniali dodawanie użytkowników. Nie wiem jak to tam działa.

Tak, w Django to będzie zupełnie inaczej. W ogóle cała obsługa tej bazy danych będzie wtedy do usunięcia.

Edytuje dane użytkownika.
Zwraca True jeśli się udało, False jeśli wystąpił błąd.
"""
try:
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Przy edycji użytkownika mogę już podać dowolne hasło i maila. Nie było weryfikacji.


class User(BaseModel):
DoesNotExist = None
username = CharField(max_length=55, unique=True)
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Zastanawiam się czy dla czytelności nie byłoby fajnie jawnie zadeklarować primary key i nazwać go np. user_id dla odróżnienia od innych id, które będą się pojawiały w bazie? Taka luźna propozycja :)

Comment on lines +1 to +5
"""
zainstalowane pakiety na moim środowisku:
peewee - ORM dla sqlite

"""
Copy link
Collaborator

@marekwocka marekwocka Nov 22, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

U mnie to przeszkadzało zainstalować pakiety z pliku requirements ;) musiałem dodać # do każdej linii

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.

4 participants