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

usuniecie instancji Listu #657

Closed
ad-m opened this issue Oct 12, 2020 · 3 comments · Fixed by #749
Closed

usuniecie instancji Listu #657

ad-m opened this issue Oct 12, 2020 · 3 comments · Fixed by #749
Labels

Comments

@ad-m
Copy link
Member

ad-m commented Oct 12, 2020

Zapoznaj się pierw z #655, aby zrozumieć kontekst zagadnienia.

Opis (co jest wymagane, wymagane implementacyjne, kryteria akceptacji):

  • Potrzebujemy stworzenie ekranu do usuwania Listu.
  • Formularz zawiera wszystkie pola dostępne w operacji api letters_delete.
  • Operacja dostępna z widoku szczegółowego Listów i z wykazu Listów
  • Rozwiązanie spójne w zakresie usuwania z innymi elementami (zob. makieta widoku szczegółowego sprawy #606)
    • Do zastanowienia się czy wykorzystamy nowy ekran, okno modalne, czy podajnik (drawer). Proponuje dla akcji takich jak usunięcie wykorzystać podajnik (drawer) na początek.
  • Back-end został zaimplementowany – jeżeli nie wiesz jak go użyć, napisz tu komentarz.

Powiązane zagadnienia:

Pomocne materiały:

@ad-m ad-m changed the title edycja instancji Listu usuniecie instancji Listu Oct 12, 2020
@michalczukm
Copy link
Contributor

Bierzemy temat z @gtktsc

@michalczukm
Copy link
Contributor

PR: #749

ad-m pushed a commit that referenced this issue Dec 31, 2020
* issue/657 remove letter instance from list

Co-authored-by: Bartosz Gryta <gtktsc@gmail.com>

* issue/657 show modal before delete letter instance

Co-authored-by: Bartosz Gryta <gtktsc@gmail.com>

* issue/657 added confirmation modal to tags delete operation

Co-authored-by: Bartosz Gryta <gtktsc@gmail.com>

Co-authored-by: Bartosz Gryta <gtktsc@gmail.com>
@michalczukm
Copy link
Contributor

Done. Można zaktualizować status ticket'a :)

@ad-m ad-m linked a pull request Dec 31, 2020 that will close this issue
@ad-m ad-m closed this as completed Dec 31, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants