Skip to content

Replace $.ajax in admin broadcast messages with axios

Clement Ho requested to merge axios-admin-message into master

What does this MR do?

Replace $.ajax in admin broadcast messages with axios. Existing specs in spec/features/admin/admin_broadcast_messages_spec.rb will cover this change.

Are there points in the code the reviewer needs to double check?

Whether I converted js.haml to json properly

Why was this MR needed?

Screenshots (if relevant)

Does this MR meet the acceptance criteria?

What are the relevant issue numbers?

https://gitlab.com/gitlab-org/gitlab-ce/issues/42397

Edited by Clement Ho

Merge request reports