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

Friendship overrides other configured permissions #228

Merged
merged 2 commits into from
Jun 6, 2024

Conversation

ChaelCodes
Copy link
Owner

Description of Feature or Issue

If the profile has befriended another profile, they are explicitly saying they want to be friends, and sharing their information - even if they normally don't allow unconfirmed users to view their profile - is ideal.

Bonus :focus works in RSpec, and documentation format will be used if only one file is run.

Checklist

  • Added/changed specs for the changes made
  • Is the linting build successful?
  • Is the test build successful?

User-Facing Changes

Unauthenticated user who is friended by a profile can view/access that profile.

If a profile has marked another profile as "friendly", then they want
to befriend that profile, and we should prioritize that over any other
permission constraints they've placed such as confirmed emails, etc.
With the exception of myself - where friends are explicitly excluded.
* Allow :focus to be used in tests
* Use documentation format automatically when one file is run
@ChaelCodes ChaelCodes force-pushed the perms-for-friends branch from 1ab3482 to 3ce2580 Compare June 6, 2024 18:25
@ChaelCodes ChaelCodes merged commit f7335dc into main Jun 6, 2024
2 checks passed
@ChaelCodes ChaelCodes deleted the perms-for-friends branch June 6, 2024 18:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant