-
Notifications
You must be signed in to change notification settings - Fork 0
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
[F003] Dashboard layout #7
base: develop
Are you sure you want to change the base?
Conversation
6db1014
to
8f29015
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We should also write some tests, like testing element visibility when user is logged in vs not logged in, the setPageTitle
helper etc
I didn't test it there because I wasn't using redux, originally redux is supposed to be in use so its gonna be tested in the next feature when redux is actually being used and authentification is handled. 😃 |
What happened
Created the dashboard layout. 👷🏾
Insight
Created UI tests using cypress for checking the header and navigation.
Proof Of Work