- Issue created by @lelkneralfaro
- 🇦🇹Austria drunken monkey Vienna, Austria
drunken monkey → made their first commit to this issue’s fork.
- Merge request !15Fix "Notification interval" field vanishing on module update → (Merged) created by drunken monkey
- 🇦🇹Austria drunken monkey Vienna, Austria
Thanks a lot for reporting this issue!
This is of course a grave problem, and very good you spotted and reported this before the 1.0 stable release!I think it’s better to discuss this in a new issue.
I managed to indeed reproduce your problem, and it seems to be about incorrect default values for the new “Notification interval” settings on the search type in some of our code (namely the one adding the configurable field to saved searches). Also, just re-saving the saved search type’s configuration form resolved the problem for me.Should be fixed in this MR – please test/review!
- Status changed to Needs review
about 1 year ago 11:24am 17 March 2024 - 🇦🇹Austria drunken monkey Vienna, Austria
Would be great if you could test/review the MR so we can resolve this issue.
Apologies for my delay on reviewing this. I will review today.
- Status changed to RTBC
about 1 year ago 5:49pm 27 March 2024 I tested your MR and the fix worked. Marking RTBC. Thank you!
-
drunken monkey →
committed 76ee19ba on 8.x-1.x
Issue #3425521 by drunken monkey: Fixed module update hiding the "...
-
drunken monkey →
committed 76ee19ba on 8.x-1.x
- Status changed to Fixed
about 1 year ago 1:50pm 29 March 2024 - 🇦🇹Austria drunken monkey Vienna, Austria
Great to hear, thanks for testing and reporting back.
Merged. Thanks again! Automatically closed - issue fixed for 2 weeks with no activity.