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

Migrate gcp-wif #5410

Merged
merged 2 commits into from
Jan 8, 2025
Merged

Migrate gcp-wif #5410

merged 2 commits into from
Jan 8, 2025

Conversation

vipin-dfe
Copy link
Contributor

Context

Changes proposed in this pull request

Enable google federated auth

Guidance to review

Data should be visible in the events table

Copy link

github-actions bot commented Jan 6, 2025

Review app deployed to https://cpd-ecf-review-5410-web.test.teacherservices.cloud

@vipin-dfe vipin-dfe force-pushed the enable_federated_auth branch from 32a40c7 to 304ca44 Compare January 8, 2025 15:07
@vipin-dfe vipin-dfe closed this Jan 8, 2025
@vipin-dfe vipin-dfe reopened this Jan 8, 2025
@vipin-dfe vipin-dfe force-pushed the enable_federated_auth branch 3 times, most recently from 8e8b44a to d2bf565 Compare January 8, 2025 15:33
@vipin-dfe vipin-dfe marked this pull request as ready for review January 8, 2025 15:34
@vipin-dfe vipin-dfe force-pushed the enable_federated_auth branch from d2bf565 to 88e650e Compare January 8, 2025 15:53
@vipin-dfe
Copy link
Contributor Author

Link to delete review app GitHub workflow - https://github.com.mcas.ms/DFE-Digital/early-careers-framework/actions/runs/12673597902

@vipin-dfe vipin-dfe force-pushed the enable_federated_auth branch from 2a96c3c to 12c33ba Compare January 8, 2025 16:51
@vipin-dfe vipin-dfe added this pull request to the merge queue Jan 8, 2025
Merged via the queue into main with commit 9f8ef9c Jan 8, 2025
29 checks passed
@vipin-dfe vipin-dfe deleted the enable_federated_auth branch January 8, 2025 17:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants