Skip to content

Add a link to switch between alpha and new Europa docs#1601

Closed
slumbering wants to merge 1 commit into
dagger:mainfrom
slumbering:docs-switcher
Closed

Add a link to switch between alpha and new Europa docs#1601
slumbering wants to merge 1 commit into
dagger:mainfrom
slumbering:docs-switcher

Conversation

@slumbering
Copy link
Copy Markdown
Contributor

The link is visible only by an exhaustive list of allowed user.

Signed-off-by: jffarge jf@dagger.io

@slumbering slumbering requested a review from gerhard February 11, 2022 14:10
@netlify
Copy link
Copy Markdown

netlify Bot commented Feb 11, 2022

✔️ Deploy Preview for devel-docs-dagger-io ready!

🔨 Explore the source changes: b105036

🔍 Inspect the deploy log: https://app.netlify.com/sites/devel-docs-dagger-io/deploys/62175a66794a8e0008e1c7d0

😎 Browse the preview: https://deploy-preview-1601--devel-docs-dagger-io.netlify.app

@slumbering slumbering requested review from aluzzardi and removed request for gerhard February 23, 2022 20:42
Signed-off-by: jffarge <jf@dagger.io>
Copy link
Copy Markdown
Contributor

@aluzzardi aluzzardi left a comment

Choose a reason for hiding this comment

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

Code looks good, but it didn't work with the preview URL

@slumbering
Copy link
Copy Markdown
Contributor Author

Code looks good, but it didn't work with the preview URL

Because we are in a none Auth mode, you probably don't have in your localstorage the following key:

user: {"permission":true,"login":"aluzzardi"}

If you add it manually, you should be able to see the link

@slumbering slumbering requested a review from aluzzardi February 25, 2022 12:40
@slumbering
Copy link
Copy Markdown
Contributor Author

@gerhard : is it still relevant ?

@slumbering
Copy link
Copy Markdown
Contributor Author

@gerhard : As we spoke today, this PR is not relevant anymore. Should we close it ?

@slumbering slumbering closed this Mar 4, 2022
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