OCA / hr-expense

Human Resources Expenses OCA modules for Odoo
GNU Affero General Public License v3.0
51 stars 103 forks source link

[15.0][MIG] hr_expense_journal #180

Closed Saran440 closed 10 months ago

yboujraf commented 1 year ago

Dear @Saran440 Don't know how to help for fixing those errors

Best Regards,

Saran440 commented 1 year ago

@yboujraf It still red because test script is not cover. you can help me to add test script and push it to this PR.

yboujraf commented 1 year ago

Dear @Saran440 Never done but if you tell me where I will double check it.

Best Regards,

Saran440 commented 1 year ago

@yboujraf Fixed, can you check it again please?

yboujraf commented 1 year ago

Is anyone could merge it ?

yboujraf commented 1 year ago

Dear @Saran440

I have installed the hr_expense_journal on my dev platform and

image

Will wait for the merge

Thanks again

yboujraf commented 1 year ago

Anybody could merge the migration ?

yboujraf commented 1 year ago

Dear @Saran440 Who could merge the migration ? I am still using the migration branch. I am expecting to use from branch 15.0. BR

yboujraf commented 1 year ago

@Saran440 , @Freni-OSI Pls, Anyone could merge this repo ? BR

Saran440 commented 1 year ago

@yboujraf if you're tested and it work. Could you approve this PR please?

yboujraf commented 11 months ago

Dear @Saran440

I confirm the PR is working for V15.0. Please would you like to merge.

Best Regards, Youssef

Saran440 commented 11 months ago

@yboujraf we need 2 person approved for merge.

May be you please approved this PR by go to Files changed > Review changes > Approve Selection_008 Selection_009

yboujraf commented 11 months ago

Dear

Done. Hope to use the released version soon and pls take a look for migration to 16.0 as well.

Best Regards Youssef


From: Saran @ Ecosoft @.> Sent: Friday, October 6, 2023 12:06 To: OCA/hr-expense @.> Cc: BY-SYSTEMS SRL Boujraf Youssef @.>; Mention @.> Subject: Re: [OCA/hr-expense] [15.0][MIG] hr_expense_journal (PR #180)

@yboujrafhttps://github.com/yboujraf we need 2 person approved for merge.

May be you please approved this PR by go to Files changed > Review changes > Approve [Selection_008]https://user-images.githubusercontent.com/20896369/273175280-b683f859-0be1-48f1-9a65-328099020aaf.png [Selection_009]https://user-images.githubusercontent.com/20896369/273175291-c30cdf68-051b-40d2-8f7e-d98cb4b8d31f.png

— Reply to this email directly, view it on GitHubhttps://github.com/OCA/hr-expense/pull/180#issuecomment-1750332867, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABO5YWWVJWLVZRRKV2QZVC3X57J3DAVCNFSM6AAAAAAX7DHLOKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONJQGMZTEOBWG4. You are receiving this because you were mentioned.Message ID: @.***>

yboujraf commented 10 months ago

Dear @Saran440

Could you check why "runboat/build" is pending ?

Best Regards

Saran440 commented 10 months ago

/ocabot merge nobump

OCA-git-bot commented 10 months ago

What a great day to merge this nice PR. Let's do it! Prepared branch 15.0-ocabot-merge-pr-180-by-Saran440-bump-nobump, awaiting test results.

OCA-git-bot commented 10 months ago

Congratulations, your PR was merged at c9e3d772cf9a7f371a95c3ffd9a6b2139d94628b. Thanks a lot for contributing to OCA. ❤️