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

feat: Add and configure Entra ID and Azure B2C. #816

Open
wants to merge 27 commits into
base: develop
Choose a base branch
from

Conversation

cafuego
Copy link
Contributor

@cafuego cafuego commented Sep 3, 2024

Refs: OPS-10529

…c client name.

Which means if you signed up with workflow A, you could not login with workflow B. Oops?

Refs: OPS-10529
… tabs.

And also modify it a bit to allow redirecting to the correct OpenID CLient URLs.

Refs: OPS-10529
…workflow-on-tabs

chore: Update config to work with the (to be updated) ocha_azure_twea…
Copy link

github-actions bot commented Sep 3, 2024

Coverage Report

Totals Coverage
Statements: 44.21% ( 1573 / 3558 )
Methods: 26.14% ( 23 / 88 )
Lines: 44.67% ( 1550 / 3470 )

Copy link

github-actions bot commented Sep 3, 2024

Build output

Composer Validate `success`

PHP Lint `success`

Docker Build `success`

Environment Setup `success`

Site Install `success`

PHP Code Sniffer `success`

Software Versions PHP 8.2.21 (cli) (built: Jul 4 2024 04:37:18) (NTS) Copyright (c) The PHP Group Zend Engine v4.2.21, Copyright (c) Zend Technologies with Zend OPcache v8.2.21, Copyright (c), by Zend Technologies with Xdebug v3.3.2, Copyright (c) 2002-2024, by Derick Rethans Composer version 2.7.8 2024-08-22 15:28:36
Drupal Logs
PHP Logs

Pusher: @cafuego, Action: `pull_request`, Workflow: `Run tests``;

@cafuego cafuego requested review from attiks and lazysoundsystem and removed request for lazysoundsystem September 3, 2024 03:07
Copy link
Contributor

@lazysoundsystem lazysoundsystem left a comment

Choose a reason for hiding this comment

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

(Untested)

Copy link
Contributor

@attiks attiks left a comment

Choose a reason for hiding this comment

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

👍

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.

3 participants