-
-
Notifications
You must be signed in to change notification settings - Fork 4
Conversation
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.
Can you also include error boundaries in this PR?
@@ -186,6 +185,10 @@ p { | |||
justify-content: center !important; | |||
} | |||
|
|||
.album-list-info { |
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.
Isn't default justified as left?
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.
No. We have made it center for /explore
which shows people name, places name etc. In center.
frontend/src/pages/Albums.js
Outdated
); | ||
}; | ||
|
||
export default Favourites; |
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.
Shouldn't this be albums
?
frontend/src/pages/Albums.js
Outdated
} | ||
`; | ||
|
||
const Favourites = () => { |
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.
Please rename it to Albums
😛
No description provided.