Skip to content

Add documentation about the identity federation concept in SCS - #72

Merged
garloff merged 2 commits into
mainfrom
feat/federation-concept
Sep 20, 2023
Merged

garloff merged 2 commits into
mainfrom
feat/federation-concept

Conversation

@reqa

@reqa reqa commented Sep 5, 2023

Copy link
Copy Markdown
Contributor

Signed-off-by: Arvid Requate <requate@univention.de>
Co-authored-by: Juan Pedro Torres <JuanP.95.torres@gmail.com>
@reqa
reqa force-pushed the feat/federation-concept branch from 487f8e5 to ac684da Compare September 6, 2023 08:54
@garloff
garloff self-requested a review September 6, 2023 10:07

@garloff garloff left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This looks good to me.
I wonder whether this should become a set of ADRs in the standards repo though.

@fkr

fkr commented Sep 20, 2023

Copy link
Copy Markdown
Member

This looks good to me. I wonder whether this should become a set of ADRs in the standards repo though.

yes, I did actually think the same when reading it.

@garloff
garloff merged commit cbf6ea1 into main Sep 20, 2023
@garloff
garloff deleted the feat/federation-concept branch September 20, 2023 10:44
@garloff

garloff commented Sep 20, 2023

Copy link
Copy Markdown
Member

Let's merge it for now -- tracking a possible move in #74.

reqa added a commit that referenced this pull request Nov 8, 2023
…#72)

Signed-off-by: Arvid Requate <requate@univention.de>
maxwolfs added a commit that referenced this pull request Nov 8, 2023
#107)

* fixup! Add documentation about the identity federation concept in SCS (#72)

Signed-off-by: Arvid Requate <requate@univention.de>

* Update identity-federation-in-scs.md

Signed-off-by: Max Wolfs <mail@maxwolfs.com>

---------

Signed-off-by: Arvid Requate <requate@univention.de>
Signed-off-by: Max Wolfs <mail@maxwolfs.com>
Co-authored-by: Max Wolfs <mail@maxwolfs.com>
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.

4 participants