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

Show "Leave group" option for open and restricted groups that user is a member of #6638

Merged
merged 1 commit into from
Oct 30, 2024

Commits on Oct 30, 2024

  1. Show "Leave group" option for open and restricted groups that user is…

    … a member of
    
    Previously the "Leave group" option was only shown for private groups on the
    basis that membership of open and restricted groups was managed by h admins.
    However first-party Hypothesis users can now create open and restricted groups
    themselves and members of these groups will see an option to leave the group on
    activity pages. Align whether the client shows the "Leave group" option with h's
    behavior.
    
    During this change it was noticed that the documentation in the code and test
    descriptions did not match the actual handling of `allowLeavingGroups`. The
    comments said that services had to explicitly set `allowLeavingGroups` to
    prevent users leaving. However the code would treat `allowLeavingGroups` as false
    if a) service configuration was present and b) the value of `allowLeavingGroups`
    was falsey (including undefined). Changing this behavior may cause issues for
    existing users of third party authorities, so this commit updates the
    documentation and tests to accurately describe the current behavior.
    
    Fixes #6637
    robertknight committed Oct 30, 2024
    Configuration menu
    Copy the full SHA
    600b3fe View commit details
    Browse the repository at this point in the history