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

patch: Add post install patches for backward compatibility #3

Merged
merged 5 commits into from
Jul 14, 2022

Conversation

ruchamahabal
Copy link
Member

@ruchamahabal ruchamahabal commented Jul 11, 2022

Added HR patches from ERPNext for backward compatibility.

@ruchamahabal ruchamahabal force-pushed the patch branch 2 times, most recently from a49c5ca to f3d478b Compare July 12, 2022 07:43
@ruchamahabal ruchamahabal force-pushed the patch branch 3 times, most recently from e2bd07b to 4908d45 Compare July 13, 2022 05:29
@codecov-commenter
Copy link

codecov-commenter commented Jul 13, 2022

Codecov Report

Merging #3 (897e30d) into develop (5113565) will decrease coverage by 0.09%.
The diff coverage is 20.00%.

@@             Coverage Diff             @@
##           develop       #3      +/-   ##
===========================================
- Coverage    71.73%   71.64%   -0.10%     
===========================================
  Files          173      174       +1     
  Lines         8576     8591      +15     
===========================================
+ Hits          6152     6155       +3     
- Misses        2424     2436      +12     
Impacted Files Coverage Δ
hrms/setup.py 0.00% <0.00%> (ø)
...lary_component_account/salary_component_account.py 100.00% <100.00%> (ø)

@ruchamahabal ruchamahabal force-pushed the patch branch 2 times, most recently from 56a811e to 9cc4703 Compare July 13, 2022 06:06
@ruchamahabal ruchamahabal merged commit 4d5ff1e into frappe:develop Jul 14, 2022
@ruchamahabal ruchamahabal deleted the patch branch July 14, 2022 06:07
scdanieli added a commit to alyf-de/hrms that referenced this pull request Nov 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants