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

Snaglist #54

Open
6 of 14 tasks
Lan2u opened this issue Jul 20, 2024 · 4 comments
Open
6 of 14 tasks

Snaglist #54

Lan2u opened this issue Jul 20, 2024 · 4 comments

Comments

@Lan2u
Copy link
Collaborator

Lan2u commented Jul 20, 2024

  • Email sent page on http://localhost:8080/send-email/owner-agreement-invite doesn't give the user a clear pathway forward and looks completely different to other pages - Suggest we get rid of the separate page and just have it indicate on the previous page that the email was sent
  • Display currently registered training sheet.
  • Allow trainers to remove/replace training sheet
  • Use memberInput everywhere
  • Filter allowed trainers for a piece of equipment to owners
  • Warn adding an owner if they haven't signed the agreement
  • Owners list on areas shows member number - names might be more useful?
  • When you add a piece of red equipment we redirect to the general equipment page. Would make more sense to go to the page for the area (or the new piece of equipment directly)
  • Warn about adding a member as trained if they haven't got a passed quiz
  • Quiz results shouldn't be shown for non-owners
  • Members should be able to see their own quiz results (both on the members page and the equipment page)
  • Declare super-user lets you add someone who is already a super-user
  • If you try and add someone as a super-user who is already a super-user then it gives an ugly error
  • There are duplicate navbars on the declare super-user page
@chromy
Copy link
Collaborator

chromy commented Jul 26, 2024

8161cac I think is sufficient for 'Display currently registered training sheet'.

@Lan2u
Copy link
Collaborator Author

Lan2u commented Jul 26, 2024

If you try and add someone as a super-user who is already a super-user then it gives an ugly error

Handled this by just avoiding it > Declare super-user lets you add someone who is already a super-user

@chromy
Copy link
Collaborator

chromy commented Jul 27, 2024

Allow trainers to remove/replace training sheet

Allowed replacement by trainers in 1cf3f67

@chromy
Copy link
Collaborator

chromy commented Aug 1, 2024

Ward ran into this one:

Email sent page on http://localhost:8080/send-email/owner-agreement-invite doesn't give the user a clear pathway forward and looks completely different to other pages - Suggest we get rid of the separate page and just have it indicate on the previous page that the email was sent

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Options
Development

No branches or pull requests

2 participants