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
Edit company interest model to include more fields #3226
Conversation
5cc5c11
to
1a10c79
Compare
Codecov ReportBase: 88.29% // Head: 88.31% // Increases project coverage by
Additional details and impacted files@@ Coverage Diff @@
## master #3226 +/- ##
==========================================
+ Coverage 88.29% 88.31% +0.01%
==========================================
Files 658 659 +1
Lines 20728 20762 +34
==========================================
+ Hits 18302 18336 +34
Misses 2426 2426
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
1a10c79
to
852f57c
Compare
@@ -53,7 +53,8 @@ def handle_create(self, instance, **kwargs): | |||
# reply is sent to them (and bedkom) | |||
recipients.append(mail_context["mail"]) | |||
if mail_context["readme"]: | |||
recipients.append(f"lederreadme@{settings.GSUITE_DOMAIN}") | |||
# recipients.append(f"lederreadme@{settings.GSUITE_DOMAIN}") |
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.
👀
@@ -53,7 +53,8 @@ def handle_create(self, instance, **kwargs): | |||
# reply is sent to them (and bedkom) | |||
recipients.append(mail_context["mail"]) | |||
if mail_context["readme"]: | |||
recipients.append(f"lederreadme@{settings.GSUITE_DOMAIN}") | |||
# recipients.append(f"lederreadme@{settings.GSUITE_DOMAIN}") | |||
recipients.append("redaktor@abakus.no") |
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.
recipients.append("redaktor@abakus.no") | |
recipients.append("redaktor@{settings.GSUITE_DOMAIN}") |
not that it matters much
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.
TBH it's fine as it is for now, so since this is needed ASAP it can be merged.
However, making enums seems like a necessary change for the future
Co-authored-by: Jonas de Luna Skulberg <jonasdls@gmail.com>
852f57c
to
b196ad8
Compare
Edited the Company Interest in according to the requested changes by "Interkom"
All the above changes are thoroughly tested though the front end and the ensomnia app