Skip to content

Commit

Permalink
Merge pull request #178 from mikerae/171-bugadmin-profile-shows-as-if…
Browse files Browse the repository at this point in the history
…-section-member

[#171] Fix User Profile dropdown
  • Loading branch information
mikerae authored Oct 10, 2023
2 parents 9d034e5 + 42de758 commit d1274b6
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 0 deletions.
2 changes: 2 additions & 0 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -288,6 +288,8 @@ The following manual tests were made:
- [see issue #170 here](https://github.com/mikerae/string-rota/issues/170)
- [Admin Register Player #172 Testing](/readme_assets/images/testing-issue172.png)
- [see issue #172 here](https://github.com/mikerae/string-rota/issues/172)
- [Fix User Profile dropdown #171 Testing](/readme_assets/images/testing-issue171.png)
- [see issue #171 here](https://github.com/mikerae/string-rota/issues/171)
#### Human Testing
No human testing was done for this MVP, but once office manager functionality, and the hiding of draft rotas is implemented, user feedback will be sort.

Expand Down
Binary file added readme_assets/images/testing-issue171.png
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
5 changes: 5 additions & 0 deletions templates/base.html
Original file line number Diff line number Diff line change
Expand Up @@ -70,12 +70,17 @@
<ul class="dropdown-menu dropdown-menu-end dropdown-menu-arrow profile">
<li class="dropdown-header">
<h6>{{user.first_name}} {{user.last_name}}</h6>
{% if user.is_superuser %}
Use your super powers wisely young Padawan...
{% elif office %}
{% else %}
{% if section %}
<p><span>Member of {{section}} Section</span></p>
{% endif %}
{% if rota_manager %}
<p><span>{{section}} Rota Manager</span></p>
{% endif %}
{% endif %}
</li>
<li>
<hr class="dropdown-divider">
Expand Down

0 comments on commit d1274b6

Please sign in to comment.