Admin stores add edit
Summary
Implement interface to create/edit stores.
Other changes
- Rename
:resource_modalturbo frame to:resource_formand add an overridable method#resource_form_framein resources_controller, that returns default frame name:resource_form. - ~UI select component: add support for
:include_blankoption. Component now will only accept array forchoices. Updated usage in a few places where a string has been passed instead of an array (user store credits)~ - split to #6279. - ~Updated layout to accommodate alerts display. Added FlashHelper to differentiate between toast flashes and alert flashes~ - split to #6280.
- ~Updated usages of
flash[:success]toflash[:notice]to be consistent~ - split to #6280.
Checklist
Check out our PR guidelines for more details.
The following are mandatory for all PRs:
- [x] I agree that my PR will be published under the same license as Solidus.
- [x] I have written a thorough PR description.
- [x] I have kept my commits small and atomic.
- [x] I have used clear, explanatory commit messages.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 86.64%. Comparing base (
2aed8c3) to head (40e76d7).
Additional details and impacted files
@@ Coverage Diff @@
## main #6228 +/- ##
==========================================
- Coverage 88.86% 86.64% -2.22%
==========================================
Files 857 520 -337
Lines 18415 11947 -6468
==========================================
- Hits 16365 10352 -6013
+ Misses 2050 1595 -455
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
:rocket: New features to boost your workflow:
- :snowflake: Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
- :package: JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.
going to split few changes into separate PRs
What's the status of this PR? It's a very large draft that I don't want to just leave sitting here.
@jarednorman it was waiting for flashes PR to be merged, so now I can move it forward