-
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
Pv testing bugdfix #3018
Pv testing bugdfix #3018
Conversation
src/Components/AoPage/AoPage.jsx
Outdated
@@ -26,7 +26,7 @@ const AoPage = () => ( | |||
<Route path="/profile/ao/availablebidders" render={() => <AvailableBidderContainer isCDO={false} isAO />} /> | |||
<Route path="/profile/ao/panelmeetingagendas/:pmID" render={() => <PanelMeetingAgendas isAO />} /> | |||
<Route path="/profile/ao/cyclemanagement" render={() => <CycleManagement isAO />} /> | |||
<Route path="/profile/ao/projectedvacancy" render={() => <ProjectedVacancy isCDO={false} isAO />} /> | |||
<Route path="/profile/ao/projectedvacancy" render={() => <ProjectedVacancy isCDO={false} isAO viewType="ao" />} /> |
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 think we should scrap the isAO prop since you have this new one
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 removed that prop in the PV components but forgot to remove it here
You new prop is a lot more descriptive than isAO, but I think the isAO prop can handle what you're aiming to do as well if you just pull that into the ProjectedVacancy components. Just leave it and delete "isAO"
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.
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.
also dont need the isCDO={false}
bit either - not used lol
No description provided.