Skip to content
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

Link header badges to Auction Result / Career Highlights #3200

Merged
merged 1 commit into from Feb 26, 2020

Conversation

lilyfromseattle
Copy link
Contributor

@lilyfromseattle lilyfromseattle commented Feb 26, 2020

PURCHASE-1782

User experience feedback: The new labels Highest auction result and Career Stage feel/look tappable – could also lead users to Auction Result / Career Highlights?

I updated the indicators so that the auction record indicator links to the auction results tab, and the career stage indicator links to the cv page.

@artsy-peril artsy-peril bot added Jira Synced Version: Patch Indicates that this PR should have a patch deploy, usually for bug fixes labels Feb 26, 2020
@zephraph zephraph added Version: Minor Indicates that this PR should have a minor deploy, usually for new features and removed Version: Patch Indicates that this PR should have a patch deploy, usually for bug fixes labels Feb 26, 2020
@zephraph
Copy link
Contributor

Looks good! I played w/ it on storybooks too. 👍

@zephraph zephraph merged commit 2c91db5 into artsy:master Feb 26, 2020
@codecov
Copy link

codecov bot commented Feb 26, 2020

Codecov Report

Merging #3200 into master will increase coverage by <.1%.
The diff coverage is 100%.

@@           Coverage Diff            @@
##           master   #3200     +/-   ##
========================================
+ Coverage    82.4%   82.4%   +<.1%     
========================================
  Files         761     761             
  Lines       11349   11349             
  Branches     2245    2245             
========================================
+ Hits         9356    9357      +1     
+ Misses       1383    1381      -2     
- Partials      610     611      +1

@artsyit
Copy link
Contributor

artsyit commented Feb 26, 2020

🚀 PR was released in v25.19.0 🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Jira Synced released Version: Minor Indicates that this PR should have a minor deploy, usually for new features
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants