Skip to content

Conversation

@nhippenmeyer
Copy link
Contributor

When using pundit we should not display links to associated resources unless the user has access to those resources.

@nhippenmeyer nhippenmeyer changed the title Hide link is user is not authorized to access resource Hide link if user is not authorized to access resource Jan 4, 2022
@nhippenmeyer nhippenmeyer force-pushed the belongs-to-index branch 5 times, most recently from 6162c36 to 279c3b4 Compare January 6, 2022 22:00
Copy link
Collaborator

@pablobm pablobm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for this. I've been meaning to provide a more comprehensive solution for a while, but this will do for now.

@pablobm pablobm merged commit da77d88 into thoughtbot:main Jan 13, 2022
@shadoath shadoath deleted the belongs-to-index branch March 11, 2025 21:05
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.

2 participants