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

Critters can see staff internal shifts #23

Closed
Corborax opened this issue Sep 4, 2024 · 1 comment · Fixed by #41
Closed

Critters can see staff internal shifts #23

Corborax opened this issue Sep 4, 2024 · 1 comment · Fixed by #41
Assignees
Labels
bug Something isn't working
Milestone

Comments

@Corborax
Copy link
Member

Corborax commented Sep 4, 2024

User Role

Attendee (Critter)

System Type

Other

Operating System

Other

OS Version

No response

Browser Details (if applicable)

No response

Steps to Reproduce

  1. Have an internal role for Staff
    grafik
  2. If a Critter wants to join the join the group, they will be added as "uncomfirmed"
    grafik

Expected Behavior

Shifts that are hidden can't be seen by members who are not confirmed members of this year.

Actual Behavior

  1. Being an uncomfirmed member of a critter type enables this Critter, still being uncomfirmed, to view internal Staff shifts from the Critty Type menu / Critty type details Shift tab
    grafik
    2.It also enables them to view the internal Staff shifts from the Shift overview
    grafik

Feature Area

Volunteer Sign-Up

If you selected "Other", please specify

No response

Severity

Critical Problem

Error Messages

No response

Workarounds Tried

No response

Network Type

None

Anything Else?

2 changes are required:

  • Hide the unconfirmed crittertypes in the shift overview / list.
  • Completely block/hide the shift tab within the critter type info if the option "Hide from shift view" is set.
@Corborax Corborax added the bug Something isn't working label Sep 4, 2024
@Corborax Corborax added this to the EF28 milestone Sep 4, 2024
@RustyBraze RustyBraze self-assigned this Sep 4, 2024
@Balenty
Copy link

Balenty commented Sep 6, 2024

Alternatively, these changes could be done (2nd change is differnet):

  1. In the shift search, you cannot filter your Crittetypes in which you are just unconfirmed (when this hide option is set).
  2. The shift tab in the Crittertype Infopage is only available when you are a full member of this Crittertype (when this hide option is set).

RustyBraze added a commit that referenced this issue Sep 7, 2024
- Fix the missing " from the German language file
- Hide/show the shift button based on Critter permission level
- Critter shifts filter is updated to match the permission level
@RustyBraze RustyBraze linked a pull request Sep 7, 2024 that will close this issue
RustyBraze added a commit that referenced this issue Sep 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants