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

DW-579: Updated email handler default bodies #80

Merged
merged 4 commits into from
Sep 13, 2022

Conversation

jekuaitk
Copy link
Contributor

@jekuaitk jekuaitk commented Sep 9, 2022

https://jira.itkdev.dk/browse/DW-579

  • Adds OS2Forms Selvbetjening module to allow setting a description for the message body section of the email handler configuration.

@jekuaitk jekuaitk requested a review from rimi-itk September 9, 2022 08:29
Copy link
Contributor

@rimi-itk rimi-itk left a comment

Choose a reason for hiding this comment

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

Godkendt under protest!

@jekuaitk jekuaitk requested a review from rimi-itk September 9, 2022 12:03
@jekuaitk jekuaitk force-pushed the feature/DW-579-update-email-handler-default-body branch from 895e61b to 6aa4c38 Compare September 9, 2022 12:34
README.md Outdated
description should be configured in the `settings.local.php` file:

```php
$config['os2forms_selvbetjening']['email_body_description'] = "Brug enten standardsvaret eller definer dit eget svar. Se <a href=\"https://os2forms.os2.eu/mail-tekster\">OS2Forms Loop</a> for andre standarder og eksempler.";
Copy link
Contributor

Choose a reason for hiding this comment

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

Use ' to simplify the description content (\ can be removed).

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Updated.

README.md Outdated
$config['os2forms_selvbetjening']['email_body_description'] = "Brug enten standardsvaret eller definer dit eget svar. Se <a href=\"https://os2forms.os2.eu/mail-tekster\">OS2Forms Loop</a> for andre standarder og eksempler.";
```

If it is not, no description is added.
Copy link
Contributor

Choose a reason for hiding this comment

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

Are you missing a “set” here, i.e. should it read “If it is not set, …”?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Updated.

description: 'Provides changes to webform email handler.'
package: 'OS2Forms'
core: 8.x
core_version_requirement: ^8 || ^9
Copy link
Contributor

Choose a reason for hiding this comment

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

Remove ^8 || – Drupal 8 is not supported anymore.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Removed.

@jekuaitk jekuaitk requested a review from rimi-itk September 9, 2022 13:09
@jekuaitk jekuaitk merged commit 14139d0 into develop Sep 13, 2022
@jekuaitk jekuaitk deleted the feature/DW-579-update-email-handler-default-body branch September 13, 2022 06:32
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.

2 participants