Skip to content
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 authentication docs to fix iron-session link. #33483

Merged
merged 3 commits into from
Jan 21, 2022
Merged

[docs] Update authentication docs to fix iron-session link. #33483

merged 3 commits into from
Jan 21, 2022

Conversation

tharakabimal
Copy link
Contributor

Switched the example links

Bug

  • Related issues linked using fixes #number
  • Integration tests added
  • Errors have helpful link attached, see contributing.md

Feature

  • Implements an existing feature request or RFC. Make sure the feature request has been accepted for implementation before opening a PR.
  • Related issues linked using fixes #number
  • Integration tests added
  • Documentation added
  • Telemetry added. In case of a feature if it's used or not.
  • Errors have helpful link attached, see contributing.md

Documentation / Examples

  • Make sure the linting passes by running yarn lint

example links updated.
Copy link
Member

@balazsorban44 balazsorban44 left a comment

Choose a reason for hiding this comment

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

Why are the two changes the opposite of each other?

The repo's README shows the current URL as https://iron-session-example.vercel.app, not the one you recommend. 🤔

@vvo could you verify which one is correct?

@vvo
Copy link
Member

vvo commented Jan 20, 2022

Hey, there the right url is https://iron-session-example.vercel.app/ yes. It was previously a different URL but I decided to link directly to the source (the iron session GitHub example).

@tharakabimal may I ask you what triggered your will to change? Thanks!

@tharakabimal
Copy link
Contributor Author

tharakabimal commented Jan 20, 2022

Hey @vvo according to the doc this is not supposed to show the loader.

image

vvo added a commit to vvo/next.js that referenced this pull request Jan 20, 2022
docs/authentication.md Outdated Show resolved Hide resolved
Copy link
Member

@vvo vvo left a comment

Choose a reason for hiding this comment

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

You can accept the request changes

Co-authored-by: Vincent Voyer <vincent@codeagain.com>
@leerob leerob changed the title Update authentication.md [docs] Update authentication docs to fix iron-session link. Jan 21, 2022
@timneutkens timneutkens merged commit d866121 into vercel:canary Jan 21, 2022
natew pushed a commit to natew/next.js that referenced this pull request Feb 16, 2022
…3483)

Co-authored-by: Vincent Voyer <vincent@codeagain.com>
Co-authored-by: Lee Robinson <me@leerob.io>
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 20, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants