-
Notifications
You must be signed in to change notification settings - Fork 1
ID-816 New Orch User Registration Flow #196
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
base: main
Are you sure you want to change the base?
Conversation
Pushed image: us-central1-docker.pkg.dev/dsp-artifact-registry/thelma/thelma:v1.0.100-40c9544 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, just need to fix the missing error handler clause and fix the failing test
"Hogwarts", "dsde", | ||
"Cambridge", "MA", "USA", | ||
"Remus Lupin", "Non-Profit") | ||
if err = opts.handleErrorWithForce(err); err != nil { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
keep this error check :)
Pushed image: us-central1-docker.pkg.dev/dsp-artifact-registry/thelma/thelma:v1.0.100-69ad458 |
Pushed image: us-central1-docker.pkg.dev/dsp-artifact-registry/thelma/thelma:v1.0.100-8fe1094 |
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## main #196 +/- ##
==========================================
+ Coverage 54.91% 54.98% +0.06%
==========================================
Files 176 176
Lines 8234 8233 -1
==========================================
+ Hits 4522 4527 +5
+ Misses 3125 3119 -6
Partials 587 587
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Pushed image: us-central1-docker.pkg.dev/dsp-artifact-registry/thelma/thelma:v1.0.100-f354db1 |
https://broadworkbench.atlassian.net/browse/ID-816
Orch now has an endpoint that both registers a user and accepts the terms of service in one go. Thelma should use it!