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

Update redirects and links for Gallery Insights content #5082

Merged
merged 2 commits into from
Feb 20, 2020
Merged

Update redirects and links for Gallery Insights content #5082

merged 2 commits into from
Feb 20, 2020

Conversation

dblandin
Copy link
Member

@dblandin dblandin commented Feb 20, 2020

  • Add specific redirects for Gallery Insights articles

    Instead of redirecting all articles in the Gallery Insights channel to the partners.artsy.net homepage, if a redirect mapping exists, redirect to specific content.

  • Redirect Gallery Insights link to /gallery-resources page

    Instead of linking Gallery Insights on the /galleries page to the partners.artsy.net home page, link to the /gallery-resources page directly.

jira 🔒 : https://artsyproduct.atlassian.net/browse/GALL-1560

Instead of redirecting all articles in the Gallery Insights channel to
the partners.artsy.net homepage, if a redirect mapping exists, redirect
to specific content.
Instead of linking Gallery Insights on the /galleries page to the
partners.artsy.net home page, link to the /gallery-resources page
directly.
@dblandin dblandin requested a review from eessex February 20, 2020 18:10
@dblandin dblandin self-assigned this Feb 20, 2020
@codecov
Copy link

codecov bot commented Feb 20, 2020

Codecov Report

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

@@           Coverage Diff            @@
##           master   #5082     +/-   ##
========================================
+ Coverage    79.2%   79.2%   +<.1%     
========================================
  Files        1242    1243      +1     
  Lines       34045   34049      +4     
  Branches     2011    2012      +1     
========================================
+ Hits        26969   26973      +4     
  Misses       6059    6059             
  Partials     1017    1017

Copy link
Contributor

@eessex eessex left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This works for me. We may want to follow up by unpublishing this content from Writer.

@dblandin dblandin merged commit 655d133 into artsy:master Feb 20, 2020
@dblandin dblandin deleted the update-gallery-insights-links branch February 20, 2020 18:59
@dblandin
Copy link
Member Author

dblandin commented Feb 20, 2020

We may want to follow up by unpublishing this content from Writer.

@eessex Sounds good! I'll create a separate ticket for that follow-up work. What are the benefits of unpublishing the content in Writer? Would unpublishing eventually remove those articles from Google search results?

@dblandin
Copy link
Member Author

FYI, here's a new Jira ticket for the follow-up work to unpublish these articles: https://artsyproduct.atlassian.net/browse/GALL-2631 🔒

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants