-
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
Update/bid tracker remaining bids message #715
Conversation
@@ -62,16 +62,16 @@ class BidTracker extends Component { | |||
render() { |
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'm trying to tell - was anything actually updated in this file, or did the line spacing just change?
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.
In the end, after being further explained the intention of the ticket, no. Before, when I added a whole alert and was counting approved bids and non-approved, yes. I guess when I removed the unnecessary code, the spacing updated.
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.
Cool, see if you can revert the changes so that the spacing is back to normal
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.
ok, i'll try
{title} has been filled | ||
Your bid on {title} is no longer applicable. |
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.
👍
…taPhase-Consulting/State-TalentMAP into update/bidTrackerRemainingBidsMessage merging with dev
Will discuss at Retro. Not sure if this is what was agreed upon.