Skip to content
This repository has been archived by the owner on Aug 16, 2024. It is now read-only.

refactor: remove OpenAM logic #121

Merged
merged 2 commits into from
Mar 19, 2024
Merged

refactor: remove OpenAM logic #121

merged 2 commits into from
Mar 19, 2024

Conversation

fibble
Copy link
Contributor

@fibble fibble commented Mar 15, 2024

Description

Removes OpenAM Logic from repos.

Related issue: VOL-2661

Before submitting (or marking as "ready for review")

  • Does the pull request title follow the conventional commit specification?
  • Have you performed a self-review of the code
  • Have you have added tests that prove the fix or feature is effective and working
  • Did you make sure to update any documentation relating to this change?

@fibble fibble changed the title Remove open am vol 2661 refactor: remove OpenAM logic Mar 15, 2024
@fibble fibble marked this pull request as ready for review March 15, 2024 14:59
Copy link
Contributor

@JoshuaLicense JoshuaLicense left a comment

Choose a reason for hiding this comment

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

Looks good! Will hold off approval until a new olcs-auth is tagged.

@JoshuaLicense JoshuaLicense merged commit 760ae2e into main Mar 19, 2024
3 of 6 checks passed
@JoshuaLicense JoshuaLicense deleted the remove-open-am-vol-2661 branch March 19, 2024 12:32
jerotire pushed a commit to dvsa/vol-app that referenced this pull request Aug 16, 2024
* refactor: Remove openAM logic - fix tests

* chore: bump `olcs-auth` to `v7.0`

---------

Co-authored-by: JoshuaLicense <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants