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

loggin-into-identity-provider+location #1847

Merged

Conversation

MichalMaler
Copy link
Contributor

Read our Contribution guide before submitting a PR.

What does this PR do?

Addind the procedure about logging in to {identity-provider} and placing it in desired destinations.

What issues does this PR fix or reference?

Specify the version of the product this PR applies to.

PR Checklist

As the author of this Pull Request I made sure that:

  • vale has been run successfully against the PR branch
  • Link checker has been run successfully against the PR branch
  • Documentation describes a scenario that is already covered by QE tests, otherwise an issue has been created and acknowledged by Che QE team
  • Changed article references are updated where they are used (or a redirect has been set up on the docs side):

Signed-off-by: Michal Maléř <mmaler@redhat.com>
Signed-off-by: Michal Maléř <mmaler@redhat.com>
@MichalMaler MichalMaler marked this pull request as ready for review February 12, 2021 12:19
@MichalMaler
Copy link
Contributor Author

VALE will be fixed by #1803

@yhontyk yhontyk self-assigned this Feb 12, 2021
Copy link

@yhontyk yhontyk left a comment

Choose a reason for hiding this comment

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

Left some comments.

…y-provider.adoc

Co-authored-by: Yana Hontyk <yhontyk@redhat.com>
@MichalMaler MichalMaler merged commit e1db6fd into eclipse-che:master Feb 12, 2021
@MichalMaler MichalMaler deleted the loggin-into-identity-provider branch February 12, 2021 14:50
@che-bot che-bot added this to the 7.27 milestone Feb 12, 2021
@themr0c themr0c modified the milestones: 7.27, 7.26.x Feb 24, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants