Conversation
GumisC4
requested review from
olios123
and removed request for
Antoni-Czaplicki
There was a problem hiding this comment.
Pull request overview
Adds a “copy quiz” capability to the app by introducing a new QuizService endpoint call and exposing it via a new action button on the quiz info card UI.
Changes:
- Added
QuizService.copyQuiz(quizId)that POSTs toquizzes/{id}/copy/. - Added a new icon button in
QuizInfoCardto trigger copying and show success/error toasts.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 4 comments.
| File | Description |
|---|---|
| src/services/quiz.service.ts | Adds a service-layer method to call the backend “copy quiz” endpoint. |
| src/components/quiz/quiz-info-card.tsx | Adds a new UI action button that calls services.quiz.copyQuiz and displays toast feedback. |
|
Hej @GumisC4, dogadaj się z Copilotem i zrobię review. |
|
Jutro zerknę i dam feedback |
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.
You can also share your feedback on Copilot code review. Take the survey.
* feat: added copy quiz feature * feat: added copy quiz feature * refactor: design and logic changes --------- Co-authored-by: Oliwier Kądziołka <71970315+olios123@users.noreply.github.com> Co-authored-by: olios123 <oliwier4444k@gmail.com>
|
Siemka, mam lekkie uwagi do tego.
|
|
Hej
|
|
masz racje ze teraz nigdzie nie ma opcji skopiowania mojego quizu, ale jeżeli taka opcje chcemy to powinna ona być w bibliotece a nie podczas robienia quizu |
|
Ja się z tym zgadzam że powinna być, tylko czy mimo to chcemy ten przycisk wewnątrz quizu dać tylko dla udostępnionych ? |
|
imo tak, tam kopiowanie nie ma wgl sensu imo |
|
imo ok imo, dodam do biblioteki a stamtąd zostawię tylko dla udostępnionych ale dodam to do #171 |
|
spoczko, to w nowym designie wtedy właśnie w bibliotece opcja do kopiowania quizu |
No description provided.