Skip to content

Dev #357

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 20 commits into from
Nov 5, 2025
Merged

Dev #357

merged 20 commits into from
Nov 5, 2025

Conversation

@TatevikGr
Copy link
Contributor

@TatevikGr TatevikGr commented Sep 4, 2025 •
edited
Loading

TatevikGr and others added 17 commits August 8, 2025 22:23
@TatevikGr @tatevikg1
Blacklist (#352)
ab6c5fd 
* Blacklisted

* user repository methods

* fix configs

* add test

* fix: phpmd

* fix: repo configs

* return a created resource

---------

Co-authored-by: Tatevik <tatevikg1@gmail.com>
@TatevikGr @tatevikg1
Subscribepage (#353)
f80edc8 
* subscriber page manager

* owner entity

* test

* ci fix

* getByPage data

---------

Co-authored-by: Tatevik <tatevikg1@gmail.com>
@TatevikGr @tatevikg1
Bounceregex manager (#354)
9d0c1db 
* BounceRegexManager

* Fix manager directory

* Prop name update admin -> adminId

---------

Co-authored-by: Tatevik <tatevikg1@gmail.com>
@TatevikGr @tatevikg1
Bounce processing command (#355)
dc99df1 
* BounceManager

* Add bounce email

* Move to the processor dir

* SendProcess lock service

* ClientIp + SystemInfo

* ProcessBouncesCommand

* ProcessBouncesCommand all methods

* BounceProcessingService

* AdvancedBounceRulesProcessor

* UnidentifiedBounceReprocessor

* ConsecutiveBounceHandler

* Refactor

* BounceDataProcessor

* ClientFactory + refactor

* BounceProcessorPass

* Register services + phpstan fix

* PhpMd

* PhpMd CyclomaticComplexity

* PhpCodeSniffer

* Tests

* Refactor

* Add tests

* More tests

* Fix tests

---------

Co-authored-by: Tatevik <tatevikg1@gmail.com>
@TatevikGr @tatevikg1
EventLog + translator (#356)
793c260 
* EventLogManager

* Log failed logins + translate messages

* weblate

* test fix

* Use translations

* Fix pipeline

* Weblate

* Deprecate DB translation table

---------

Co-authored-by: Tatevik <tatevikg1@gmail.com>
@TatevikGr @tatevikg1
Access level check (#358)
936cabc 
* OwnableInterface

* PermissionChecker

* Check related

* Register service + test

* Style fix

---------

Co-authored-by: Tatevik <tatevikg1@gmail.com>
@TatevikGr @tatevikg1
Message processor (#359)
fa42258 
* MessageStatusEnum

* Status validate

* Embargo check

* IspRestrictions

* IspRestrictions

* SendRateLimiter

* UserMessageStatus

* Refactor

* RateLimitedCampaignMailer

* RateLimitedCampaignMailerTest

* Rate limit initialized from history

* Check maintenance mode

* Max processing time limiter

---------

Co-authored-by: Tatevik <tatevikg1@gmail.com>
@TatevikGr @tatevikg1
Exceptions + translations (#361)
fd51731 
* MissingMessageIdException

* MailboxConnectionException

* BadMethodCallException (style fix)

* Translate user facing messages

* Translate

* Translate PasswordResetMessageHandler texts

* Translate SubscriberConfirmationMessageHandler texts

* MessageBuilder exceptions

* BlacklistEmailAndDeleteBounceHandler

* BlacklistEmailHandler - AdvancedBounceRulesProcessor

* AdvancedBounceRulesProcessor

* BounceStatus

* CampaignProcessor

* UnidentifiedBounceReprocessor

* Style fix

* Test fix

* ConsecutiveBounceHandler

---------

Co-authored-by: Tatevik <tatevikg1@gmail.com>
@tatevikg1
Fix autowiring
6e65b28
@tatevikg1
Reset subscriber bounce count
da72f1f
@tatevikg1
Install RssFeedBundle
feea299
@TatevikGr @tatevikg1
Update import logic, add dynamic attribute repository (#362)
175dacb 
* Skip password and modified fields while import, do not subscribe blacklisted users

* DefaultConfigProvider

* Use ConfigProvider in ProcessQueueCommand

* Use ConfigProvider in SubscriberCsvImporter + send email

* Rename paramProvider

* Test fix

* AttributeValueProvider for dynamic tables

* Update: config provider

* Translations in default configs

* Email with messageHandler

* Style fix

* PhpCs fix

* Fix configs

* replace list names

* Add tests + fix

* Add more tests + fix handler

---------

Co-authored-by: Tatevik <tatevikg1@gmail.com>
@TatevikGr @tatevikg1
Refactor import add subscriber history (#363)
9adb0ec 
* Add blacklisted stat to import result

* Add history

* Add translations

* addHistory for unconfirmed

* Refactor

* Add changeSetDto

* Do not send email on creating without any list

* Flush in controller

* Add test

---------

Co-authored-by: Tatevik <tatevikg1@gmail.com>
@TatevikGr @tatevikg1
Em flush (#364)
6932106 
* createSubscriberList

* ProcessQueueCommand

* Dispatch CampaignProcessorMessage for processing

* Fix tests

* Fix style

* CleanUpOldSessionTokens

* Move bounce processing into the Bounce folder

* delete/remove

* Remove hardcoded TatevikGrRssBundle mapping

* Fix configs

* Add sync

* Fix: DQL in MessageRepository.php

* PhpMD

* SubscriberBlacklistService, SubscriberBlacklistManager

* AdministratorManager

* BounceManager

* SendProcessManager

* TemplateManager

* SubscribePageManager

* Fix: tests

* BounceManager

* rest of flushes

* save

* fix test

* CouplingBetweenObjects 15

---------

Co-authored-by: Tatevik <tatevikg1@gmail.com>
@tatevikg1
Add test
93547aa
@tatevikg1
Update: docs
987873e
@TatevikGr @tatevikg1
Migrations (mysql psql) (#366)
938b090 
* OnlyOrmTablesFilter

* Current

* Admin

* Init migration

* In progress

* Fix mapping

* Fix tests

* Migrate

* Separate MySql

* Use psql

* Rename indexes

* PostgreSqlPlatform

* MySqlSqlPlatform rename indexes

* Fix: cs

* Fix: test configs

* Add migration template

* PR agent (#365)

* .coderabbit.yaml

This reverts commit 2246e49.

---------

Co-authored-by: Tatevik <tatevikg1@gmail.com>

* rename template

* After review 0

* After review 1

* Fix MySql migrations

* After review 2

---------

Co-authored-by: Tatevik <tatevikg1@gmail.com>
@coderabbitai
Copy link

coderabbitai bot commented Oct 31, 2025 •
edited
Loading

Important

Review skipped

More than 25% of the files skipped due to max files limit. The review is being skipped to prevent a low-quality review.

157 files out of 300 files are above the max files limit of 100. Please upgrade to Pro plan to get higher limits.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch dev

Comment @coderabbitai help to get the list of available commands and usage tips.

TatevikGr and others added 3 commits November 5, 2025 11:04
@TatevikGr @tatevikg1
Import by foreign key (#367)
5231d0a 
* Import with a foreign key

---------

Co-authored-by: Tatevik <tatevikg1@gmail.com>
@TatevikGr @tatevikg1
Insert initial admin command (#368)
323e67f 
* Update: codeRabbit configs

* Update: Use command for initial admin insert


---------

Co-authored-by: Tatevik <tatevikg1@gmail.com>
@tatevikg1
Update pull request template
f2d12bc
@TatevikGr TatevikGr merged commit 033aa54 into main Nov 5, 2025
8 checks passed
@TatevikGr TatevikGr deleted the dev branch November 5, 2025 09:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@TatevikGr @tatevikg1