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(admin): fix toggle and tristate input elements not working correctly #278

Merged
merged 1 commit into from
Nov 13, 2024

Conversation

EdieLemoine
Copy link
Contributor

@EdieLemoine EdieLemoine commented Oct 10, 2024

  • Fixes "disable delivery options" in product settings not being saved

@EdieLemoine EdieLemoine requested a review from a team as a code owner October 10, 2024 15:03
Copy link

codecov bot commented Oct 10, 2024

Codecov Report

Attention: Patch coverage is 0% with 15 lines in your changes missing coverage. Please review.

Project coverage is 48.06%. Comparing base (165c91b) to head (b176ed4).
Report is 9 commits behind head on main.

✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
...backend/admin/src/components/pdk/PsToggleInput.vue 0.00% 14 Missing ⚠️
...ckend/admin/src/components/pdk/PsTriStateInput.vue 0.00% 1 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##               main     #278      +/-   ##
============================================
- Coverage     51.96%   48.06%   -3.90%     
+ Complexity      690      561     -129     
============================================
  Files           152      148       -4     
  Lines          4191     3728     -463     
  Branches         55       55              
============================================
- Hits           2178     1792     -386     
+ Misses         1958     1881      -77     
  Partials         55       55              
Flag Coverage Δ
backend 76.97% <ø> (-1.07%) ⬇️
frontend 0.00% <0.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.


🚨 Try these New Features:

@EdieLemoine EdieLemoine force-pushed the fix/fix-product-settings-saving branch from 90a9165 to 5a866d3 Compare October 10, 2024 15:08
@EdieLemoine EdieLemoine changed the title fix/fix product settings saving fix(admin): fix toggle and tristate input elements not working correctly Oct 10, 2024
@EdieLemoine EdieLemoine marked this pull request as draft October 10, 2024 15:09
@myparcel-bot myparcel-bot bot added the approved (Auto) label Oct 11, 2024
@EdieLemoine EdieLemoine changed the base branch from beta to main October 16, 2024 11:35
@EdieLemoine EdieLemoine marked this pull request as ready for review October 24, 2024 11:20
@EdieLemoine EdieLemoine force-pushed the fix/fix-product-settings-saving branch from 5a866d3 to fef0909 Compare October 24, 2024 11:20
@EdieLemoine EdieLemoine enabled auto-merge October 24, 2024 11:20
@EdieLemoine EdieLemoine disabled auto-merge October 24, 2024 11:21
@myparcel-bot myparcel-bot bot force-pushed the fix/fix-product-settings-saving branch 4 times, most recently from da57535 to b01decf Compare October 30, 2024 13:49
@myparcel-bot
Copy link
Contributor

myparcel-bot bot commented Oct 30, 2024

✅ Build successful!

You can find a link to the downloadable artifact below.

Name Link
Commit b176ed4
Logs https://github.com/myparcelnl/prestashop/actions/runs/11724472878
Download https://github.com/myparcelnl/prestashop/suites/30617801043/artifacts/2157945901

@myparcel-bot myparcel-bot bot force-pushed the fix/fix-product-settings-saving branch from b01decf to 430ad55 Compare October 30, 2024 18:08
- Fixes "disable delivery options" in product settings not being saved

Needs myparcelnl/js-pdk#223
@myparcel-bot myparcel-bot bot force-pushed the fix/fix-product-settings-saving branch from 430ad55 to b176ed4 Compare November 7, 2024 13:58
@EdieLemoine EdieLemoine added this pull request to the merge queue Nov 13, 2024
Merged via the queue into main with commit c8832fd Nov 13, 2024
6 of 7 checks passed
@EdieLemoine EdieLemoine deleted the fix/fix-product-settings-saving branch November 13, 2024 13:20
myparcel-bot bot added a commit that referenced this pull request Nov 20, 2024
## [4.2.0](v4.1.0...v4.2.0) (2024-11-20)

### 🐛 Bug Fixes

* **admin:** fix toggle and tristate input elements not working correctly ([#278](#278)) ([c8832fd](c8832fd)), closes [myparcelnl/js-pdk#223](myparcelnl/js-pdk#223)
* **orders:** export correct recipient name ([#277](#277)) ([883dcf4](883dcf4)), closes [#276](#276)
* **upgrade:** always execute module installation on upgrade ([#271](#271)) ([78b3657](78b3657))

### ✨ New Features

* **backend:** add download logs action ([#275](#275)) ([7bb1349](7bb1349)), closes [myparcelnl/pdk#288](myparcelnl/pdk#288) [myparcelnl/js-pdk#221](myparcelnl/js-pdk#221)
* **checkout:** show carriers only for countries they ship to ([#273](#273)) ([e13b6a2](e13b6a2))
* **deps:** upgrade @myparcel-pdk/checkout to v1.2.0 ([0b314e9](0b314e9))
* **deps:** upgrade myparcelnl/pdk to v2.47.1 ([ffe8ce4](ffe8ce4))
@myparcel-bot
Copy link
Contributor

myparcel-bot bot commented Nov 20, 2024

🎉 This PR is included in version 4.2.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

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

Successfully merging this pull request may close these issues.

4 participants