Skip to content

Commit

Permalink
[#175] Hide CRUD Buttons in Rota for Office
Browse files Browse the repository at this point in the history
Show CRUD Rota buttons for Admin and Rota manager
Hide CRUD Rota buttons for Player and Orch manager
	modified:   string_rota/templates/string_rota/rota.html

Add Testing for Hide CRUD Buttons in Rota for Office to README
	modified:   README.md
	new file:   readme_assets/images/testing-issue175.png
  • Loading branch information
mikerae committed Oct 10, 2023
1 parent d1274b6 commit 3cc4d36
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 1 deletion.
2 changes: 2 additions & 0 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -290,6 +290,8 @@ The following manual tests were made:
- [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)
- [Fix CRUD Buttons for Office #175 Testing](/readme_assets/images/testing-issue175.png)
- [see issue #175 here](https://github.com/mikerae/string-rota/issues/175)
#### 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-issue175.png
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
2 changes: 1 addition & 1 deletion string_rota/templates/string_rota/rota.html
Original file line number Diff line number Diff line change
Expand Up @@ -130,7 +130,7 @@ <h5 class="text-info">Please contact your Rota Manager</h5>

<!-- Rota Form Section-->
{% if user.is_authenticated %}
{% if user.is_staff %}
{% if user.is_superuser %}
<div class="col-12 text-center">
<a href="{% url 'add_sp_office' project.slug seating_plan.id section.id %}" class="btn btn-primary">Add
Player to Seating Plan</a>
Expand Down

0 comments on commit 3cc4d36

Please sign in to comment.