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.

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

[ADD][16.0] module hr_timesheet_sheet_quick_add #704

Draft
wants to merge 1 commit into
base: 16.0
Choose a base branch
from
Draft

[ADD][16.0] module hr_timesheet_sheet_quick_add #704

wants to merge 1 commit into from

Conversation

bealdav
Copy link
Member

@bealdav bealdav commented Aug 16, 2024 •
edited
Loading

@bealdav bealdav mentioned this pull request Aug 17, 2024
Merged
@bealdav bealdav force-pushed the quick-add branch 2 times, most recently from 8a521c7 to 83322ef Compare August 18, 2024 17:15
@bealdav bealdav marked this pull request as ready for review August 18, 2024 17:20
@pedrobaeza pedrobaeza added this to the 16.0 milestone Aug 19, 2024
@pedrobaeza
Copy link
Member

This should be called hr_timesheet_sheet_quick_add.

@bealdav bealdav changed the title [ADD][16.0] module hr_timesheet_quick_add [ADD][16.0] module hr_timesheet_sheet_quick_add Aug 19, 2024
@dora-jurcevic
Copy link

Great feature !
Maybe it would be interesting to define the period of the timesheets that we want to display in the Timesheet Candidates field . In some cases I immagine 20 days would show too many candidates.
Thx !

@bealdav bealdav marked this pull request as draft September 3, 2024 11:57
@bealdav
Copy link
Member Author

bealdav commented Sep 3, 2024

Needs to be fixed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@PaulGoubert PaulGoubert PaulGoubert approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
16.0
Development

Successfully merging this pull request may close these issues.
4 participants
@bealdav @pedrobaeza @dora-jurcevic @PaulGoubert