Skip to content

[17.0][ADD] hr_attendance_report #222

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
Open
wants to merge 2 commits into
base: 17.0
Choose a base branch
from
Open

[17.0][ADD] hr_attendance_report #222

wants to merge 2 commits into from

Conversation

Arubaru86
Copy link

This module allows users to generate employee attendance reports in PDF and Excel formats within the Odoo Attendance system. It provides a wizard for filtering and exporting attendance data with customizable date ranges, employees, and departments.

Features:

  • Generates PDF and Excel attendance reports.
  • Filters attendance data by employee, department, month, and year.
  • Displays detailed information, including check-in, check-out, and worked hours.
  • Supports multiple employee and department selections.
  • Integrates into the HR Attendance menu and enhances reporting capabilities.

Impact:

  • Simplifies the process of generating attendance reports for HR teams.
  • Improves the efficiency of reporting for payroll or compliance purposes.
  • Provides a more streamlined and accurate tracking of employee attendance data.
  • Enhances visibility and reporting capabilities across departments.

@pedrobaeza
Copy link
Member

I would remove the _pdf suffix in the module, moreover exporting also to Excel.

@Arubaru86 Arubaru86 changed the title [ADD] Module for generating employee attendance reports in PDF and Ex… [17.0][ADD] hr_attendance_report May 2, 2025
@Arubaru86 Arubaru86 force-pushed the 17.0-hr_attendance_report_pdf branch from 39d7265 to e9da676 Compare June 4, 2025 07:09
javierizaca
javierizaca approved these changes Jun 4, 2025
View reviewed changes
Copy link

@javierizaca javierizaca left a comment

Choose a reason for hiding this comment

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

LGTM

@Arubaru86 Arubaru86 force-pushed the 17.0-hr_attendance_report_pdf branch 3 times, most recently from 1ca346e to d11e949 Compare June 9, 2025 10:33
Tardo
Tardo suggested changes Jun 9, 2025
View reviewed changes
Copy link
Member

@Tardo Tardo left a comment

Choose a reason for hiding this comment

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

Come on! we are getting closer and closer to have it ready :)! for my part, the code review...

@Arubaru86 Arubaru86 requested a review from Tardo June 9, 2025 11:53
@Arubaru86 Arubaru86 force-pushed the 17.0-hr_attendance_report_pdf branch 4 times, most recently from 19ac236 to 7487cfc Compare June 10, 2025 09:08
Tardo
Tardo approved these changes Jun 11, 2025
View reviewed changes
Copy link
Member

@Tardo Tardo left a comment

Choose a reason for hiding this comment

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

LGTM

@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). 🤖

@Arubaru86
Copy link
Author

Arubaru86 commented Jun 23, 2025 •
edited
Loading

Hi @pedrobaeza @yajo 👋 This PR has the approved label and has been open for more than 5 days. Could you please take a look and merge it when possible?

Thanks in advance! 😊

@pedrobaeza pedrobaeza added this to the 17.0 milestone Jun 24, 2025
@pedrobaeza
Copy link
Member

One of the approvals must be a PSC: https://github.com/OCA/odoo-community.org/blob/master/website/Contribution/CONTRIBUTING.rst#review

marcelsavegnago
marcelsavegnago approved these changes Jun 24, 2025
View reviewed changes
Copy link
Member

@marcelsavegnago marcelsavegnago left a comment

Choose a reason for hiding this comment

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

LGTM

pedrobaeza
pedrobaeza reviewed Jun 24, 2025
View reviewed changes
@pedrobaeza
Copy link
Member

pedrobaeza commented Jun 25, 2025 •
edited
Loading

You have to squash the commits. Instead of doing it in 2 steps, you can amend the commits while working on it (that's for the future cases).

@Arubaru86 Arubaru86 requested a review from pedrobaeza June 25, 2025 09:23
@Arubaru86 Arubaru86 closed this Jun 25, 2025
@Arubaru86 Arubaru86 force-pushed the 17.0-hr_attendance_report_pdf branch from f8a6414 to e251a6f Compare June 25, 2025 12:23
@Arubaru86 Arubaru86 reopened this Jun 25, 2025
@Arubaru86 Arubaru86 force-pushed the 17.0-hr_attendance_report_pdf branch from c09d93a to 62cc150 Compare June 25, 2025 12:56
@Arubaru86 Arubaru86 force-pushed the 17.0-hr_attendance_report_pdf branch from eba852c to 27949d9 Compare June 25, 2025 14:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@pedrobaeza pedrobaeza Awaiting requested review from pedrobaeza

3 more reviewers

@Tardo Tardo Tardo approved these changes

@marcelsavegnago marcelsavegnago marcelsavegnago approved these changes

@javierizaca javierizaca javierizaca approved these changes

Reviewers whose approvals may not affect merge requirements
Assignees
No one assigned
Labels
approved ready to merge
Projects
None yet
Milestone
17.0
Development

Successfully merging this pull request may close these issues.
6 participants
@Arubaru86 @pedrobaeza @OCA-git-bot @Tardo @marcelsavegnago @javierizaca