- Issue created by @lauriii
- Merge request !426#3489354 Connect the "Publish" button with the entity update controller → (Merged) created by lauriii
- 🇫🇮Finland lauriii Finland
This is currently missing test coverage and error handling. I guess it would be worth considering how extensive test coverage should we write, given that this is a temporary step.
- First commit to issue fork.
-
jessebaker →
committed a4373e94 on 0.x authored by
lauriii →
Issue #3489354 by lauriii, jessebaker, larowlan: Connect the "Publish"...
-
jessebaker →
committed a4373e94 on 0.x authored by
lauriii →
- 🇬🇧United Kingdom jessebaker
I've taken the pragmatic approach to merge this as it stands. The code in this MR is a step forward that allows a user to save albeit in a fairly unpolished way from a UX and code quality standpoint. However, it is not intended to be the final iteration of the Publish workflow so I think it's worth having merged for now despite those shortcomings.
- 🇧🇪Belgium wim leers Ghent 🇧🇪🇪🇺
+1 for decisive pragmatism, and acknowledging it with explicit names conveying the throwaway nature! 👏
Automatically closed - issue fixed for 2 weeks with no activity.
