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

[11.0][MIG] hr_holidays_notify_employee_manager #446

Merged
merged 5 commits into from
Jun 6, 2018
Merged

[11.0][MIG] hr_holidays_notify_employee_manager #446

merged 5 commits into from
Jun 6, 2018

Conversation

tarteo
Copy link
Member

@tarteo tarteo commented May 17, 2018

Depends on #445

MiquelRForgeFlow and others added 4 commits May 17, 2018 10:47
@MiquelRForgeFlow @tarteo
[MIG] hr_holidays_notify_employee_manager: Migration to 10.0
5d7da7f
@MiquelRForgeFlow @tarteo
[FIX] add company_id in view
9e83112
@oca-transbot @tarteo
OCA Transbot updated translations from Transifex
ee91b72 
OCA Transbot updated translations from Transifex

OCA Transbot updated translations from Transifex
@tarteo
[MIG] hr_holidays_notify_employee_manager: Migration to 11.0
95c3865 
[MIG] hr_holidays_notify_employee_manager: Migration to 11.0
@pedrobaeza pedrobaeza added this to the 11.0 milestone May 17, 2018
@pedrobaeza pedrobaeza mentioned this pull request May 17, 2018
Closed
71 tasks
MiquelRForgeFlow
MiquelRForgeFlow approved these changes May 30, 2018
View reviewed changes
Copy link
Contributor

@MiquelRForgeFlow MiquelRForgeFlow left a comment

Choose a reason for hiding this comment

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

Please @tarteo , rebase the PR

MiquelRForgeFlow
MiquelRForgeFlow approved these changes Jun 5, 2018
View reviewed changes
Copy link
Contributor

@MiquelRForgeFlow MiquelRForgeFlow left a comment

Choose a reason for hiding this comment

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

That's a bad rebase, but ok because 🍏 . The PR should be merged using the "rebase and merge" option.

@tarteo
Copy link
Member Author

tarteo commented Jun 5, 2018 •
edited
Loading

@mreficent I'm sorry. Thanks for the review.

@JordiBForgeFlow JordiBForgeFlow merged commit 0bae9dd into OCA:11.0 Jun 6, 2018
sambarros pushed a commit to sambarros/hr that referenced this pull request Jul 26, 2018
@leemannd
Merge pull request OCA#446 from TelmoSenseFly/BSSFL-531
d17e570 
[BSSFL-531] Company logo
Mraimou pushed a commit to camptocamp/hr that referenced this pull request Dec 4, 2020
merge OCA#446 bso_subscription_cease fix
0baad9f
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@astirpe astirpe astirpe approved these changes

@MiquelRForgeFlow MiquelRForgeFlow MiquelRForgeFlow approved these changes

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

Successfully merging this pull request may close these issues.

None yet
6 participants
@tarteo @astirpe @MiquelRForgeFlow @pedrobaeza @JordiBForgeFlow @oca-transbot