Show /contributions link in navbar when p2p enabled #821
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Why
We want to show the /contributions link in navbar when p2p enabled because it's a key part of the p2p workflow
Pre-Merge Checklist
What
Screenshots of current code:
Logged out with P2P
Logged out without P2P
Logged in
How
We're adding a prop passed to Vue that indicates if P2P is on or not, and then letting Vue show or hide the button
Testing
There are some JS tests since this is mostly a JS change
Next Steps and Concerns
We talked some about rearranging these. Our concerns were:
Because of all these concerns, we agreed to revisit how to balance them later
Accessibility
We did not do accessibility testing for this. It seems unlikely to me that it would change the accessibility from the previous code
Security
This may reveal the link URL in the JS to someone who is not logged in. That said, that URL will still have the Devise checks, so the additional insecurity is only removing obscurity.