Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Feature/configure download template #77

Merged
merged 5 commits into from
Nov 16, 2023
Merged

Feature/configure download template #77

merged 5 commits into from
Nov 16, 2023

Conversation

cgoIT
Copy link
Contributor

@cgoIT cgoIT commented Nov 15, 2023

This small change makes the used download template configurable in the settings of the placeholder form field

Copy link
Member

@Toflar Toflar left a comment

Choose a reason for hiding this comment

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

Hey, thanks a lot for the PR! Love it! I've noticed a few coding style issues and wondered why the CI chain is not running properly anymore. I've fixed this in main now so can I ask you to rebase? The GitHub CI chain should then tell you where the coding style does not match the rest of the project 😊

@cgoIT
Copy link
Contributor Author

cgoIT commented Nov 16, 2023

Hi @Toflar,

I will do that. Although I am surprised that there are problems or errors. I'm actually pretty sure that I ran the cs-fixer script locally. But I'll see what the CI job says.

@cgoIT cgoIT closed this Nov 16, 2023
…ate' into feature/configure-download-template

* origin/feature/configure-download-template:
  Update the CI chain (#78)

# Conflicts:
#	src/EventListener/LoadFormFieldListener.php
#	src/Step/FileParameterBag.php
@cgoIT cgoIT reopened this Nov 16, 2023
@Toflar Toflar merged commit 0b55f8d into terminal42:main Nov 16, 2023
14 checks passed
@Toflar
Copy link
Member

Toflar commented Nov 16, 2023

Thanks a lot for this new feature! Released as 5.1.0 😎 https://github.com/terminal42/contao-mp_forms/releases/tag/5.1.0

@cgoIT cgoIT deleted the feature/configure-download-template branch November 16, 2023 09:45
@cgoIT
Copy link
Contributor Author

cgoIT commented Nov 16, 2023

thx a lot for this wonderful extension and your fast reply

# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants