discourse/app
Martin Brennan 8a89a77248
FIX: Discard empty bundles for reviewables (#30121)
Followup c7e471d35a

It is currently possible to add a bundle (which is a collection
of actions used for a dropdown on the client) for a reviewable
via actions.add_bundle and then never add any actions to it.

This causes the client to explode, as seen in the referenced
commit, because of the way our store expects to resolve objects
referenced by ID that are passed down by the serializer, which
then causes Ember to have an unrecoverable render error.

Fixing this on the serializer level is not really possible because
of all the ActiveModel::Serializer magic that serializes
objects by ID reference when doing things like has_many.
`Reviewable#actions_for` is a better place to do this anyway,
because this is the main location where the bundles and actions
are built for every action via the serializer.
2024-12-05 15:41:13 +10:00
..
assets FIX: Move granted badges to the end of the post header. (#30095) 2024-12-05 09:22:03 +11:00
controllers UX: Improve error when trying to edit globally shadowed setting (#30092) 2024-12-04 13:41:32 +10:00
helpers DEV: Drop WithServiceHelper 2024-09-05 09:58:20 +02:00
jobs DEV: Drop DISCOURSE_LIVE_SLOTS_SIDEKIQ_LIMIT (#29920) 2024-11-26 07:21:14 +11:00
mailers UX: Use a dropdown for SSL mode for group SMTP (#27932) 2024-07-18 10:33:14 +10:00
models FIX: Discard empty bundles for reviewables (#30121) 2024-12-05 15:41:13 +10:00
serializers FEATURE: Show when a badge has been granted for a post (#29696) 2024-12-03 13:43:27 +11:00
services UX: Improve error when trying to edit globally shadowed setting (#30092) 2024-12-04 13:41:32 +10:00
views DEV: unsilence deprecation warning for old Font Awesome icons (#30028) 2024-12-03 10:28:39 +08:00