Skip to content

The admin displays the received question from the server. - #3

Merged
JanaDrazkova merged 6 commits into
mainfrom
jd_show_question
Mar 18, 2026
Merged

The admin displays the received question from the server.#3
JanaDrazkova merged 6 commits into
mainfrom
jd_show_question

Conversation

@JanaDrazkova

Copy link
Copy Markdown
Contributor

The admin displays the recieved question from the server.

@JanaDrazkova

Copy link
Copy Markdown
Contributor Author

This pull request also relates to Issue #4 – Admin receives a question. However, since the issue is linked to the quiz-server repository, it cannot be linked to this pull request. Maybe we should duplicate the issue in this repository as well?

@JanaDrazkova JanaDrazkova linked an issue Mar 5, 2026 that may be closed by this pull request
Comment thread src/quiz_admin/__main__.py
Co-authored-by: Martin Zelený <martin.zeleny@volny.cz>
verutrix
verutrix previously approved these changes Mar 12, 2026
Comment thread src/quiz_admin/__main__.py
Comment thread src/quiz_admin/__main__.py Outdated
Comment thread src/quiz_admin/__main__.py Outdated

@dslavicek dslavicek left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Just fix the type and it's fine.

Note: this introduces duplicity in printing questions as we discussed.

@JanaDrazkova
JanaDrazkova requested a review from dslavicek March 16, 2026 11:20

@dslavicek dslavicek left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM

@ZelenyMartin ZelenyMartin left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I tried it. It works and it's a great improvement! Thanks!

@JanaDrazkova
JanaDrazkova merged commit a4aaa66 into main Mar 18, 2026
2 checks passed
@JanaDrazkova
JanaDrazkova deleted the jd_show_question branch March 18, 2026 20:04
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.

[Clone] Admin receives a question

4 participants