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
Refactor: AuthProvider supports_sign_out #1462
Conversation
8048f39
to
2fe3888
Compare
Just an early thought, feel free to disregard -- maybe instead of an explicit field (causing the test/migration to fail), maybe a calculated field (property) based on the presence or absence of the sign out button text? |
Yeah, that will make things simpler 👍 |
c83203e
to
2264100
Compare
@@ -139,7 +139,7 @@ def load_data(app, *args, **kwargs): | |||
|
|||
veteran_auth_provider = AuthProvider.objects.create( | |||
sign_in_button_label=_("eligibility.buttons.veteran.signin"), | |||
sign_out_button_label=_("eligibility.buttons.veteran.signout"), |
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.
Remove this line from django.po
too?
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.
I think we could use a couple quick Model tests for this property, but I won't block for it.
a2d2047
93c85f5
to
a2d2047
Compare
Rebased to resolve merge conflicts |
|
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.
🚀
Closes #1456