AcclaroInc / craft-translations

Easily manage your Craft CMS website translations and drive global growth with simplified translation workflows.
https://plugins.craftcms.com/translations
Other
9 stars 12 forks source link

added check for already merged file from reverting back via callback #487

Closed bhupeshappfoster closed 8 months ago

bhupeshappfoster commented 8 months ago

Pull Request

Description:

Fixed file status reverting back to ready for review when its applied and we receive an order callback.

Related Issue(s):

Changes Made:

[List the changes made in this pull request to address the issue or implement the feature.]

Added:

Changed:

-

Deprecated:

-

Removed:

-

Fixed:

-

Security:

-

Testing:

[Describe the testing performed to ensure the changes are functioning as expected.]

Quality Assurance (QA):

Resources:

Screenshots (if applicable): [Include any relevant screenshots to visually demonstrate the changes.]

Wrapping up

Checklist:

Additional Notes: [Include any additional notes, considerations, or context that may be relevant.]

bhupeshappfoster commented 8 months ago

I see a bug in this one we need to allow getting quote and needs approval status too when processing files. Will be addressed soon.

MerrittBoydston commented 8 months ago

Hi @sidedwards and @shnsumit can you please review this PR when you have a moment? Thank you