_content/solutions: fix Uber case study link and description - #373
Open
mranney-dd wants to merge 1 commit into
Open
_content/solutions: fix Uber case study link and description#373mranney-dd wants to merge 1 commit into
mranney-dd wants to merge 1 commit into
Conversation
The eng.uber.com domain was retired and eng.uber.com/aresdb/ now returns 404 with no redirect, so both the case study card on /solutions/ and the Uber logo in the "Companies using Go" grid on the home page pointed at a dead page. Point them at the article's current location. Also drop a leftover editorial annotation from the description, which was rendered verbatim on /solutions/ as "AresDB [,written in Go,]", and fix "GPU-power" in the title to match the article's own "GPU-Powered".
|
Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA). View this failed invocation of the CLA check for more information. For the most up to date status, view the checks section at the bottom of the pull request. |
Author
|
/check-cla |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The
eng.uber.comdomain has been retired andhttps://eng.uber.com/aresdb/now returns 404 with no redirect. Two places on the site link to it:
/solutions/(
_content/index.md, which uses.linkfor entries withinLandingPageGrid: true)Both currently send visitors to a dead page. This points them at the
article's current location,
https://www.uber.com/blog/aresdb/, whichserves the same post ("Introducing AresDB: Uber's GPU-Powered Open Source,
Real-time Analytics Engine"). I used the locale-neutral path rather than the
/US/en/variant that/en-US/blog/aresdb/redirects to, to avoid pinningthe link to one locale.
Two small fixes to the same front matter while here:
descriptioncontained an unapplied editorial annotation,AresDB [,written in Go,]._content/solutions/case-studies.mdrenders{{.description}}verbatim, so those brackets and stray commas werevisible on the published page.
titleread "GPU-power analytics engine"; the linked article's own titleis "GPU-Powered".
Tested with
go test ./.... Two failures reproduce identically on masterand are unrelated to this change:
cmd/golangorgTestAllreports a brokenlink to
/LICENSE, andcmd/screentestrequires Google default credentials.