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

Fix objectSupport collides with Duration plugin - issue #2027 #2038

Merged
merged 2 commits into from Aug 30, 2022

Conversation

BePo65
Copy link
Contributor

@BePo65 BePo65 commented Aug 24, 2022

Using the plugins objectSupport and Duration in one file creates false result in .add and .subtract when extend(Duration) comes before extend(objectSupport) - see issue #2027.

Fixed this problem and added tests to avoid regression.

while fixing, an issue wit the handling of subtract appeared and was
fixed too.
@codecov
Copy link

codecov bot commented Aug 24, 2022

Codecov Report

Merging #2038 (3e20494) into dev (6c2a517) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##               dev     #2038   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files          181       181           
  Lines         2074      2080    +6     
  Branches       544       547    +3     
=========================================
+ Hits          2074      2080    +6     
Impacted Files Coverage Δ
src/plugin/objectSupport/index.js 100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@iamkun iamkun merged commit c9370ea into iamkun:dev Aug 30, 2022
@iamkun
Copy link
Owner

iamkun commented Aug 30, 2022

THX

iamkun pushed a commit that referenced this pull request Oct 21, 2022
## [1.11.6](v1.11.5...v1.11.6) (2022-10-21)

### Bug Fixes

* add BigIntSupport plugin ([#2087](#2087)) ([f6dce48](f6dce48))
* Fix objectSupport collides with Duration plugin - issue [#2027](#2027) ([#2038](#2038)) ([c9370ea](c9370ea))
@iamkun
Copy link
Owner

iamkun commented Oct 21, 2022

🎉 This PR is included in version 1.11.6 🎉

The release is available on:

Your semantic-release bot 📦🚀

@BePo65 BePo65 deleted the fix/issue-2027 branch October 21, 2022 04:20
ohsory1324 pushed a commit to ohsory1324/dayjs that referenced this pull request Dec 20, 2023
## [1.11.6](iamkun/dayjs@v1.11.5...v1.11.6) (2022-10-21)

### Bug Fixes

* add BigIntSupport plugin ([iamkun#2087](iamkun#2087)) ([f6dce48](iamkun@f6dce48))
* Fix objectSupport collides with Duration plugin - issue [iamkun#2027](iamkun#2027) ([iamkun#2038](iamkun#2038)) ([c9370ea](iamkun@c9370ea))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants