-
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
Client dashboard - Language History TM-2290 #1706
Conversation
…te with real data
I'll take a look at resolving conflicts |
@@ -79,7 +81,6 @@ const ProfileDashboard = ({ | |||
<BoxShadow className="usa-width-one-whole user-dashboard-section favorites-section"> | |||
<SavedSearches /> | |||
</BoxShadow> | |||
{ isBidder() && favoritesContainer() } |
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.
i don't think i meant to remove this. Will check
@@ -120,46 +121,56 @@ const ProfileDashboard = ({ | |||
</div> | |||
} | |||
{ | |||
isPublic && showClassifications && !userClassificationsHasErrored && |
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.
Broke out isPublic logic to encapsulate all the other checks - deleted lines 139
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.
🍬 👍
WIP - still need to remove fake data and integrate with WS endpoint. Haven't checked if we are already mapping the languages that are already coming through (also should double check that they STILL come through on client response).
Possible merge conflcit with elizabeth's favorites update on client dashboard