Skip to content

Conversation

TatevikGr
Copy link
Contributor

@TatevikGr TatevikGr commented Oct 1, 2025

Summary

  • add DynamicListAttrRepository to get data from dynamic tables (that has user attributes)
  • add AttributeValueProvider to resolve user attribute values from DynamicListAttrRepository
  • add default config provider
  • add config provider (to get configs from config repository or default config provider)
  • add UserPersonalizer (to replace user related placeholders with user data)
  • add SubscriptionConfirmationMessageHandler to send confirmation email to subscribed users after csv import
  • update csv import logic

Unit test

Are your changes covered with unit tests, and do they not break anything?

You can run the existing unit tests using this command:

vendor/bin/phpunit tests/

Code style

Have you checked that you code is well-documented and follows the PSR-2 coding
style?

You can check for this using this command:

vendor/bin/phpcs --standard=PSR2 src/ tests/

Other Information

If there's anything else that's important and relevant to your pull
request, mention that information here. This could include benchmarks,
or other information.

If you are updating any of the CHANGELOG files or are asked to update the
CHANGELOG files by reviewers, please add the CHANGELOG entry at the top of the file.

Thanks for contributing to phpList!

@TatevikGr TatevikGr force-pushed the send-email-after-import-by-config branch from ccdce7f to 94fe61c Compare October 2, 2025 12:03
@TatevikGr TatevikGr force-pushed the send-email-after-import-by-config branch from 94fe61c to fc6ba92 Compare October 3, 2025 10:16
@TatevikGr TatevikGr force-pushed the send-email-after-import-by-config branch from 4fe287b to eda1465 Compare October 6, 2025 09:59
@TatevikGr TatevikGr merged commit 175dacb into dev Oct 8, 2025
8 of 9 checks passed
@TatevikGr TatevikGr deleted the send-email-after-import-by-config branch October 8, 2025 07:14
@TatevikGr TatevikGr mentioned this pull request Oct 8, 2025
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