Skip to content
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.

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

[17.0][MIG] hr_attendance_rfid #204

Merged
merged 29 commits into from
Feb 12, 2025

Conversation

ThiagoMForgeFlow
Copy link
Contributor

@@ -4,7 +4,7 @@

{
"name": "HR Attendance RFID",
"version": "16.0.1.0.1",

Choose a reason for hiding this comment

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

17.0.1.0.0

@ThiagoMForgeFlow ThiagoMForgeFlow force-pushed the 17.0-mig-hr_attendance_rfid branch from 874f4f6 to 1bf3710 Compare January 9, 2025 09:16
Copy link

@LoisRForgeFlow LoisRForgeFlow left a comment

Choose a reason for hiding this comment

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

LGTM 👍

Copy link

@Neitherkx Neitherkx left a comment

Choose a reason for hiding this comment

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

Everything seems correctly migrated to Odoo 17.

Copy link
Member

@etobella etobella left a comment

Choose a reason for hiding this comment

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

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 17.0-ocabot-merge-pr-204-by-etobella-bump-nobump, awaiting test results.

OCA-git-bot added a commit that referenced this pull request Feb 3, 2025
Signed-off-by etobella
@OCA-git-bot
Copy link
Contributor

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

@OCA-git-bot
Copy link
Contributor

@etobella your merge command was aborted due to failed check(s), which you can inspect on this commit of 17.0-ocabot-merge-pr-204-by-etobella-bump-nobump.

After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red.

HviorForgeFlow and others added 17 commits February 3, 2025 15:35
[UPD] Update hr_attendance_rfid.pot
[UPD] README.rst

[UPD] Update hr_attendance_rfid.pot
[UPD] README.rst

[UPD] Update hr_attendance_rfid.pot

[ADD] icon.png
Currently translated at 100.0% (7 of 7 strings)

Translation: hr-12.0/hr-12.0-hr_attendance_rfid
Translate-URL: https://translation.odoo-community.org/projects/hr-12-0/hr-12-0-hr_attendance_rfid/fr/
Currently translated at 100.0% (7 of 7 strings)

Translation: hr-12.0/hr-12.0-hr_attendance_rfid
Translate-URL: https://translation.odoo-community.org/projects/hr-12-0/hr-12-0-hr_attendance_rfid/es/
[UPD] Update hr_attendance_rfid.pot

[UPD] README.rst

Update translation files

Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: hr-attendance-13.0/hr-attendance-13.0-hr_attendance_rfid
Translate-URL: https://translation.odoo-community.org/projects/hr-attendance-13-0/hr-attendance-13-0-hr_attendance_rfid/
[UPD] README.rst

[14.0][MIG] hr_attendance_rfid

[UPD] Update hr_attendance_rfid.pot

[UPD] README.rst
If not, the unique constraint raises when duplicating employee.

[UPD] Update hr_attendance_rfid.pot

[UPD] README.rst

hr_attendance_rfid 14.0.1.1.0

Update translation files

Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: hr-attendance-14.0/hr-attendance-14.0-hr_attendance_rfid
Translate-URL: https://translation.odoo-community.org/projects/hr-attendance-14-0/hr-attendance-14-0-hr_attendance_rfid/
lubusax and others added 11 commits February 3, 2025 15:35
[UPD] Update hr_attendance_rfid.pot

[UPD] README.rst
A ValidationError has to be converted to a string explicitly, otherwise an error is produced if is used as a string.

hr_attendance_rfid 15.0.1.0.1
Translated using Weblate (Italian)

Currently translated at 100.0% (7 of 7 strings)

Translation: hr-attendance-15.0/hr-attendance-15.0-hr_attendance_rfid
Translate-URL: https://translation.odoo-community.org/projects/hr-attendance-15-0/hr-attendance-15-0-hr_attendance_rfid/it/

[UPD] README.rst

Update translation files

Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: hr-attendance-15.0/hr-attendance-15.0-hr_attendance_rfid
Translate-URL: https://translation.odoo-community.org/projects/hr-attendance-15-0/hr-attendance-15-0-hr_attendance_rfid/
- Switch to setUpClass for avoiding repeat the same setup for each test.
- Include context keys for avoiding mail operations overhead.

[BOT] post-merge updates
Currently translated at 100.0% (7 of 7 strings)

Translation: hr-attendance-15.0/hr-attendance-15.0-hr_attendance_rfid
Translate-URL: https://translation.odoo-community.org/projects/hr-attendance-15-0/hr-attendance-15-0-hr_attendance_rfid/es/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: hr-attendance-16.0/hr-attendance-16.0-hr_attendance_rfid
Translate-URL: https://translation.odoo-community.org/projects/hr-attendance-16-0/hr-attendance-16-0-hr_attendance_rfid/
in a way RFID devices can handle it
@LoisRForgeFlow LoisRForgeFlow force-pushed the 17.0-mig-hr_attendance_rfid branch from 1bf3710 to 9592027 Compare February 3, 2025 14:35
@LoisRForgeFlow
Copy link

Rebased

@etobella
Copy link
Member

etobella commented Feb 3, 2025

Tests are failing. Can you test it?

@ThiagoMForgeFlow ThiagoMForgeFlow force-pushed the 17.0-mig-hr_attendance_rfid branch from 0b205ea to 3e0a023 Compare February 12, 2025 09:48
@LoisRForgeFlow
Copy link

@etobella could you try again? @ThiagoMForgeFlow has rebased and fixed the issues

@etobella
Copy link
Member

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 17.0-ocabot-merge-pr-204-by-etobella-bump-nobump, awaiting test results.

@OCA-git-bot
Copy link
Contributor

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

@OCA-git-bot OCA-git-bot merged commit 2834186 into OCA:17.0 Feb 12, 2025
7 checks passed
@LoisRForgeFlow LoisRForgeFlow deleted the 17.0-mig-hr_attendance_rfid branch February 12, 2025 12:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.