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

Feat: Allow to set filename for template (FIXED) #21

Merged
merged 6 commits into from
Jul 2, 2024

Conversation

spectrewall
Copy link
Contributor

Added filename parameter to allow developers to set filename in template message.

Work based on PR: #8 but as it seens to be forgotten, i took the liberty to add what was asked.

fiveone-digital and others added 5 commits May 27, 2023 18:29
- Add default filename when the given param is empty;
- Adjusts the tests\Component\DocumentTest.php to expect the filename param;
- Adjusts the tests\WhatsAppTemplateTest.php to expect the filename param;
@aalbarca
Copy link
Collaborator

aalbarca commented Apr 4, 2024

Hi @spectrewall this PR is blocked due an unsuccesful pipeline: https://github.styleci.io/analyses/yrmB3o?utm_campaign=vcs-integration-link&utm_medium=referral&utm_source=github-build-link

You can fix this error running php cs fixer or fix manually the style issues.

@aalbarca aalbarca added the invalid This doesn't seem right label Apr 4, 2024
@spectrewall
Copy link
Contributor Author

Hi @aalbarca, Sorry for being late on this, but a just fixed the error.

@aalbarca aalbarca merged commit 0cb435f into netflie:main Jul 2, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
invalid This doesn't seem right
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants