-
Notifications
You must be signed in to change notification settings - Fork 394
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
docs: update bad undo-add links #1823
Conversation
p.s. should I also add a temporary redirect Ivan? |
p.p.s. hey @jeremydesroches, how would we inspect how much traffic |
nah, I think we are fine here CI checks are still broken ^^ |
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.
CI is broken
Sorry my bad... Typo 🤦 BTW these new checks against the review app are great! Thanks @shcheklein @rogermparent |
The trick is to go into performance and then filter by the url. This link should get you there on Search Console. |
I see. Yeah I discovered that recently on Search Console actually. But this tells me how many impressions and clicks it gets, as well as the search queries, which is useful (screenshot below). But I was wondering even more general than that: the net traffic it gets, so should probably check in Google Analytics instead? |
Yes. Usually the clicks will line up pretty well with sessions (for organic), but try this view in Analytics. |
Oh cool, thanks @jeremydesroches. Do you know if there's a way to add more than one page? I'd like to overlay that one with the new URL, dvc.org/doc/user-guide/how-to/update-tracked-files to see how smooth the transition was (there's a redirect form the previous one to this one). |
Yep, try this report in Analytics. You can also use compare in Search Console. I tried out the original link and there doesn't seem to be a working redirect at the moment. It's good that the intent is to have a redirect, and I assume that will be fixed. This is actually a pretty important thing to handle for SEO purposes, so I'll just spell out what should happen in cases like this:
Hope this helps @jorgeorpinel! |
By the way, to produce that report: Find or search for the urls you want in the table, then check them off and select "Plot Rows". |
Sweet! Looks like that search box takes regexps too, which is useful to find typos in links out there. I just fixed 2 old links that now cause 404s in old GitHub comments based on this:
Page Compare on SC is also good but you can only have 2. |
Actually we completely flopped on this this time. Not a big deal since we're talking about 100 visits/week but totally agree with those steps. We usually setup the redirects (which get removed later at some point) but we never reindex on SC — good tip, thanks. I would add a step: to find any old links we have control of and update them, so the redirect can be removed later on. We already do this in all the docs of course, since it's easy to grep the old URL, but they're also common in old GH comments, for example (maybe some 3rd party blogs we wrote?) which tend to not get updated. |
Nice @jorgeorpinel! Yes, you can do some amazing things with regexp in GA. Another spot I've used them is with Filters... easy to get carried away, but if you ever find visits coming in with strange event/channel/tag data, a regexp filter can clean everything up nicely. Only for data going forward though.
Yes, SC is pretty limited... BUT you can view SC data in GA if you enable the connection! Maybe you can get the best of both... 🙂 |
Probably no harm done, as long as it's redirected relatively soon. The sitemap is likely being read daily, so new URLs will get picked up easily (e.g., a new blog post or doc will get indexed almost immediately). When URLs change it's a bit different: the 301 redirect assigns the previous URL's ranking to the new URL so it takes its place in the index. Without a redirect the new URL might be recognized as the old page, but could actually "start over". The old URL can hang around in the index for quite a while, too.
Yes, a good practice if the scope of your content allows for it. It's definitely better to have a well-maintained redirects file than hundreds of historical entries — but it happens. 😒 |
Thanks for the tips about Filters and on linking SC and GA @jeremydesroches, will check it out.
Well, the change happened weeks ago already and as you may have noticed in the merged PR #1705 (review) where I mentioned you, there was some damage done 😓 but if you want (and it can be good so you learn this) feel free to setup the redirect in redirects-list.json with a small PR. |
per https://github.com/iterative/dvc.org/runs/1179565776