Skip to content

Replace $.get in single file diff with axios

Clement Ho requested to merge axios-get-single-file-diff into master

What does this MR do?

Replace $.get in single file diff with axios

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

  • Existing specs should be covered by spec/features/expand_collapse_diffs_spec.rb
  • I removed the spec that tested ajaxUri because there is not a good way to do that with axios

Why was this MR needed?

jQuery ajax => Axios

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 Filipa Lacerda

Merge request reports

Loading