-
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
Fix bid tracker URL #863
Fix bid tracker URL #863
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.
🍨 👍
(<ClosedAlert | ||
title={BID_TITLE} | ||
date={bid.closed_date} | ||
id={bid.id} |
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.
Do we still need to send in id
?
/> | ||
); | ||
break; | ||
case DECLINED_PROP: | ||
overlayClass = CLASS_CLOSED; | ||
overlayContent = <DeclinedAlert bureau={position.bureau} id={bid.id} />; | ||
overlayContent = <DeclinedAlert bureau={position.bureau} id={bid.id} bidIdUrl={bidIdUrl} />; |
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.
Same as other comment: id
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.
Good catch, removed. Re-running tests now
* dev: Remove unecessary data Update tests and fix props Update tests and fix condensed button Bump style-loader from 1.2.0 to 1.2.1 Map correct state to props for bid tracker with actions and remove unused vars Change border inside of register overlay to white Add shorthands Add registerHandshakePosition to mdp Allow CDOs to export a client's bid list to CSV Bump postcss-flexbugs-fixes from 4.2.0 to 4.2.1 Add styling for conditional rendering of the overlay to be full width of container below 1450px in order to avoid long name collisions Shorten button in condensed form Remove tooltip on tracker for cdo view of register hs Add bid tracker styling for new overlay alert Bump eslint-loader from 4.0.1 to 4.0.2 Bump eslint-loader from 4.0.0 to 4.0.1 Check CDO role from the roles array, instead of the user profile object Add reducers # Conflicts: # src/Components/BidTracker/BidTrackerCard/BidTrackerCard.jsx # src/Components/BidTracker/OverlayAlert/OverlayAlert.jsx
Fixes the "Go to Bid Tracker" button from the condensed bid tracker when a CDO is looking at a client's public profile. Resolves a bug where the the CDO was directed to their own bid tracker instead of the client's.