[Admin][Product] fix properties edit action in admin panel part of the issue #5857

[Astr0surf3r]

Summary

I have just downloaded the gem to fix the issue #5857 and I found in admin panel an error in Property Types (in your demo the error appear as well)

After I added a new one property and no issue came up clicking in the + Add new button of the /admin/properties page

but after I saved it

I couldn't edit it because an error appear

so to fix this I add the show action in the Spree::Admin::PropertiesController and create an rspec file I hope this can help your great work

• I have written a thorough PR description.

• I have kept my commits small and atomic.

• I have used clear, explanatory commit messages.

• ✅ I have added automated tests to cover my changes.

• 📸 I have attached screenshots to demo visual changes.

[tvdeyen]

Thanks. Would you mind to update the link instead. We do not have show actions anywhere. We only ever have edit actions. By updating the link to the edit route it should be fixed

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 89.54%. Comparing base (b1849a5) to head (fc26130).

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #5885   +/-   ##
=======================================
  Coverage   89.54%   89.54%           
=======================================
  Files         783      783           
  Lines       17994    17994           
=======================================
  Hits        16113    16113           
  Misses       1881     1881           

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

[Astr0surf3r]

Thanks. Would you mind to update the link instead. We do not have show actions anywhere. We only ever have edit actions. By updating the link to the edit route it should be fixed

@tvdeyen done! 6ce9444

[tvdeyen]

@Astr0surf3r please use rebase to update your branch as mentioned in the contribution guidelines

[Astr0surf3r]

c05d55a

@Astr0surf3r please use rebase to update your branch as mentioned in the contribution guidelines

@tvdeyen I hope now the PR is cleaner than before ... if need something else to do (I have read the guidelines) let me know please I really appreciate all your tips and messages to give a hand to your great project

NOTE: now the PR is out-of-date with the base branch for this reason launched an Update Branch command