Skip to content

Remove vue_broadcast_messages feature flag

Alex Pennells requested to merge force-vue-broadcast-messages into master

What does this MR do and why?

Related: #385789 (closed)

Now that the Vue app has reached feature parity, we can safely remove the old HAML views. Remove the vue_broadcast_messages feature flag and all the old views along with it

Screenshots

This video shows off the basic workflow now that the feature is on by default. If you'd like to test the functionality against the HAML app, you'll need to use the master branch to do so. role_targeted_broadcast_messages is the other feature shown in the video, toggling it will show/hide the Target Roles setting for broadcast messages

How to set up and validate locally

  1. Navigate to the broadcast messages view: http://gdk.test:3000/admin/broadcast_messages
  2. Add/Update/Delete messages to test functionality
  3. Test with role_targeted_broadcast_messages toggled on/off for full coverage

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Edited by Alex Pennells

Merge request reports

Loading