Skip to content

Conversation

@aa-ahmed-aa
Copy link

@aa-ahmed-aa aa-ahmed-aa commented Jul 10, 2025

  • this PR is adding the support for sylius v2.1
  • add Arabic translation support
Screenshot 2025-07-10 at 7 27 05 AM

tested manually with with fresh installation of sylius 2.1

composer.json Outdated
]
},
"repositories": {
"bitbag-blacklist-plugin": {
Copy link
Member

Choose a reason for hiding this comment

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

please remove

Copy link
Author

Choose a reason for hiding this comment

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

done

Copy link
Member

@christopherhero christopherhero Jul 11, 2025

Choose a reason for hiding this comment

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

Thanks @aa-ahmed-aa ;)
I have launched gh actions (please check and fix)

Also, please leave only one or two commits, one for updating the composer and the other adding translations.

(So that you exclude from the branch any changes that change the url to the repository)

Copy link
Author

@aa-ahmed-aa aa-ahmed-aa Jul 12, 2025

Choose a reason for hiding this comment

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

sylius 2.1.0 added support to ORM 3 here for this the change in the BitBag\SyliusBlacklistPlugin\Repository\FraudSuspicionRepository is needed
Screenshot 2025-07-13 at 1 54 18 AM

so besides the changes in composer.json and the translation yml file the update for the FraudSuspicionRepository and FraudSuspicionRepositoryInterface is needed.

@aa-ahmed-aa aa-ahmed-aa force-pushed the support_sylius_v2.1 branch from baa4fed to ed04197 Compare July 11, 2025 19:28
@aa-ahmed-aa aa-ahmed-aa force-pushed the support_sylius_v2.1 branch from 1ad13cf to 6e1e316 Compare July 13, 2025 21:24
…h PrePersistEventArgs in the prePersist method and its specifications for improved event handling.
… Update methods to improve element selection and handling of empty collections, enhancing robustness and user experience.
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