liferay.design icon indicating copy to clipboard operation
liferay.design copied to clipboard

Shared Drive Migration Broke All Drive Links

Open plhnk opened this issue 3 years ago • 8 comments

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:

  1. cmd + shift + f search in VS Code for https://drive.google.com
  2. Click on any of the links (except for the links at /blueprints/resources/logo-files
  3. See 404

Expected behavior A drive folder should appear

Recommended steps to fix

  1. Perform the search in the "To Reproduce" section
  2. Go file-by-file, check the link to verify its broken
  3. 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

plhnk avatar Jul 29 '21 05:07 plhnk

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 avatar Nov 17 '21 06:11 plhnk

@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 avatar Nov 17 '21 17:11 victorvalle

@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.

plhnk avatar Nov 17 '21 17:11 plhnk

Please, let me know the outcome and how I can help.

victorvalle avatar Nov 17 '21 19:11 victorvalle

@victorvalle I searched into our shared folder and linked to the document that had most similarities with the topic on each specific page.

leolohmann avatar Nov 17 '21 19:11 leolohmann

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 avatar Nov 17 '21 19:11 victorvalle

@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

plhnk avatar Nov 17 '21 19:11 plhnk

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

plhnk avatar Nov 17 '21 23:11 plhnk