Skip to content

Comments

token exchange addition#69

Merged
mrtn78 merged 3 commits intodevelopfrom
Token_Exhange
Jan 10, 2025
Merged

token exchange addition#69
mrtn78 merged 3 commits intodevelopfrom
Token_Exhange

Conversation

@mrtn78
Copy link
Collaborator

@mrtn78 mrtn78 commented Nov 11, 2024

solves part is #63

@mrtn78 mrtn78 requested review from fterpstra and sanderke November 11, 2024 16:11
@mrtn78 mrtn78 self-assigned this Nov 11, 2024
@mrtn78 mrtn78 requested a review from TheBonheurs December 2, 2024 14:05
@sanderke sanderke linked an issue Jan 7, 2025 that may be closed by this pull request
7 tasks
@sanderke
Copy link
Member

sanderke commented Jan 9, 2025

Wijzigingen

  • ${\textsf{\color{red}↓ Verwijderd ↓}}$
  • ${\textsf{\color{green}↕ Vervangen ↕}}$
  • ${\textsf{\color{orange}↑ Toegevoegd ↑}}$

Copy link
Member

@TheBonheurs TheBonheurs left a comment

Choose a reason for hiding this comment

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

image

Dit is niet per se een change die in deze PR is doorgevoerd maar wel benieuwd naar de achterliggende gedachte (zie comments Alexander).

Deze changes zijn verder goed! Approved.

@mrtn78 mrtn78 merged commit bd685c0 into develop Jan 10, 2025
5 checks passed
@mrtn78 mrtn78 deleted the Token_Exhange branch January 10, 2025 15:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Development

Successfully merging this pull request may close these issues.

Use case: Relatie met SAML en eHerkenning / SSOOnRijk

3 participants