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

[#1567] Modified contact-forms to send new subject_code/onderwerp field to e-Suite #788

Merged
merged 2 commits into from
Oct 3, 2023

Conversation

Bartvaderkin
Copy link
Contributor

@Bartvaderkin Bartvaderkin commented Sep 26, 2023

This covers both contact-forms: the configurations each get a new field with a code that gets sent to e-Suite/OpenKlant

@Bartvaderkin Bartvaderkin marked this pull request as ready for review September 26, 2023 12:51
@Bartvaderkin Bartvaderkin requested review from vaszig and pi-sigma and removed request for vaszig September 26, 2023 12:52
@@ -507,6 +501,11 @@ def register_by_email(self, form, recipient_email):
def register_by_api(self, form, config: OpenKlantConfig):
assert config.has_api_configuration()

try:
ztc = ZaakTypeConfig.objects.filter_case_type(self.case.zaaktype).get()
except ObjectDoesNotExist:
Copy link
Contributor

Choose a reason for hiding this comment

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

Does filter_case_type always return a single object?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It does. It returns a queryset so it can be chained with other filters and situations that want a queryset (eg: filter_enabled_for_case_type() and filter_questions_enabled_for_case_type())

Copy link
Member

@alextreme alextreme left a comment

Choose a reason for hiding this comment

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

Discussed and this looks good, will need testing with the Enschede eSuite Contactmomenten API

@alextreme alextreme merged commit d048749 into develop Oct 3, 2023
@alextreme alextreme deleted the feature/1567-esuite-klant-subject branch October 3, 2023 08:19
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.

3 participants