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: welcome page for external users with terms and conditions VOL-5664 #419

Merged
merged 3 commits into from
Nov 15, 2024

Conversation

ilindsay
Copy link
Contributor

@ilindsay ilindsay commented Oct 29, 2024

Description

Welcome page now appears for external users to agree terms and conditions.

The following will need merging and dependencies bumped, before unit tests will pass:
olcs-transfer

Other related PRs:
olcs-common
olcs-etl

Related issue: VOL-5664

@ilindsay ilindsay requested review from fibble and a team as code owners October 29, 2024 09:23
@ilindsay ilindsay force-pushed the VOL-5664 branch 2 times, most recently from ecec3d4 to 43760b7 Compare November 5, 2024 12:39
@ilindsay ilindsay changed the title feat: VOL-5664 welcome page now appears for external users to agree terms and conditions feat: welcome page for external users with terms and conditions VOL-5664 Nov 12, 2024
@ilindsay ilindsay changed the title feat: welcome page for external users with terms and conditions VOL-5664 feat: welcome page for external users with terms and conditions VOL-5664 Nov 12, 2024
@ilindsay ilindsay force-pushed the VOL-5664 branch 4 times, most recently from 019720e to 423ac3d Compare November 15, 2024 00:15
fibble
fibble previously approved these changes Nov 15, 2024
@sdh100shaun sdh100shaun self-requested a review November 15, 2024 15:57
sdh100shaun
sdh100shaun previously approved these changes Nov 15, 2024
@ilindsay ilindsay dismissed stale reviews from sdh100shaun and fibble via 629adc0 November 15, 2024 17:07
@sdh100shaun sdh100shaun merged commit e4a5102 into main Nov 15, 2024
27 checks passed
@sdh100shaun sdh100shaun deleted the VOL-5664 branch November 15, 2024 17:11
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