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

Fix username in header (all user types) #36

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

Conversation

rlaiola
Copy link
Contributor

@rlaiola rlaiola commented Sep 25, 2023

Reference Issues

The proposed approach addresses the UI issue reported at #35.

What does this implement/fix?

This pull request includes a code fix that shows the username (instead of the user full name) in the header.

How to test it?

git clone --branch test_fix_username_header https://github.com/rlaiola/boca.git
cd boca
# Launch application (it may take a while)
docker compose up -d
# Open a web browser and visit the URL http://localhost:8000/boca.
# First, create and activate a BOCA contest (user: system | password: boca).
# Then, login as admin (user: admin | password: boca).
# To stop the application:
docker compose down

Other Comments

Feel free to suggest other approaches to address the problem.

@rlaiola rlaiola changed the title Fix username in header (all users types) Fix username in header (all user types) Sep 25, 2023
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.

1 participant