[16.0][ADD] hr_planning_resources #1
Conversation
|
|
||
| def has_working_hours(start_dt, end_dt): | ||
| """ | ||
| Returns `True` if there are any working days between `start_dt` and `end_dt`. |
There was a problem hiding this comment.
Should be imperative style https://stackoverflow.com/a/54868794
Same in previous docstring
There was a problem hiding this comment.
Hi, thank you for the work!
I did a small functional exploration on the interface.
- maybe a small doc on how to use the module would be nice
- I went on a project and clicked on "assign planning", there is a confirmation popup, and then nothing. What is this supposed to be doing ?
|
@antoniodavid |
Hi, could you explain the purpose of this feature? |
Thanks for the advice!!! |
|
@antoniodavid @FernandoRomera why the split? I don't get it. |
[16.0][ADD] hr_planning_resources [REF][16.0] hr_planning_resources: Refactoring app [16.0][ADD] hr_planning_resources: Add readme [16.0][ADD] hr_planning_resources: Add test-requirements Revert "[16.0][ADD] hr_planning_resources: Add readme" This reverts commit 4073ee9. [16.0][IMP] hr_planning_resources: Refactoring app
|
I also don't think it's necessary to split it since the scope of the module is clear |
I think you can edit and mention this is not really needed anymore. |
Fixed!!!! |
No description provided.