Restrict app access to anonymous users #31
Merged
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.
Description
Whenever an anonymous user visits the site, they only see a prompt message and a
Sign Inbutton. In the navbar, onlyHomeandAboutlinks are visible.Changes in the code
A new
Aboutview was added to the app. It is accessible from the navbar and has a routing set up inApp.jsx. inLayout.jsxthe header was adjusted and now displays the app logo and aSign In/Sign Outbutton. To the navbar inLayout.jsxa link forAboutsection was added. Links in navbar and content inHomeview are rendered conditionally based on whether the user is logged in or not.Related Issue
closes #30
Acceptance Criteria
Aboutview is accessible from the navbarType of Changes
enhancementUpdates
Homepage when logged out
Homepage when logged in
Testing Steps / QA Criteria
npm startHomeandAbout should be accessible