liferay.design
liferay.design copied to clipboard
Shared Drive Migration Broke All Drive Links
Describe the bug
Any drive.google.com
links that we were referencing broke when we changed from the standard drive to a shared drive (even though the subfolders all remained the same).
To Reproduce Steps to reproduce the behavior:
-
cmd + shift + f
search in VS Code forhttps://drive.google.com
- Click on any of the links (except for the links at
/blueprints/resources/logo-files
- See 404
Expected behavior A drive folder should appear
Recommended steps to fix
- Perform the search in the "To Reproduce" section
- Go file-by-file, check the link to verify its broken
- Replace the broken link w/the correct one
Note — you'll need to make sure that the permissions to the folder you're sharing are set to "everyone at Liferay.com can comment".
cc @naoki-evan-hisamoto @abelhancock probably know where most of these folders are
thanks @leolohmann! great effort!
@drakonux can you take a look at #1046 — I left a bunch of comments and tagged you in a few, some of the docs linked in Lexicon weren't clear to me, just wanted to confirm w/you
@MiriamManso can you check my comment in the pr as well — in your article on research libraries some of the links were going to permissioned files in our internal drive
I'll be merging this PR first thing tomorrow morning (for me) — if you have changes that you'd like to have here, please either send them in a PR to Leo's fork, or you can comment in here and I can put them in.
If you can't look until later, or can't send info before I merge — that's not a big deal, can always do it later :)
@plhnk I've only checked 3 links in Lexicon pages and all of them are wrong. Is there something automatic that was not applied well? You guys did it manually? If so, I need to review everything form and send a PR that fixes everything manually.
@victorvalle no it was manual — it's not possible to do it automatically, it's just our best guess. This has raised another issue that I've pinged @drakonux about — I'm thinking that it will be better to address the underlying issue (i.e. why are we adding links to google docs, and now involving confluence) than put effort into fixing the links one by one as they have been broken for some time and no one has said anything.
Please, let me know the outcome and how I can help.
@victorvalle I searched into our shared folder and linked to the document that had most similarities with the topic on each specific page.
GDrive Lexicon folder and Lexicon site structure are quite similar. Just following both trees many things should match. But in that case, I would do it. It will be far easy for me.
@victorvalle if its going to take more than 15 minutes to do it and send the PR to Leo's branch — do it, if not — IMHO its not worth it
Note — leaving this issue open — if anyone is able to tackle this in the future, please check the PR reviews in #1046 for details from @victorvalle and @drakonux