Skip to content

Replace jQuery in app/assets/javascripts/projects/star.js file

Daniel Schömer requested to merge quatauta/gitlab-contrib:363000-replace-jquery-in-app-assets-javascripts-projects-star-js into master

What does this MR do and why?

This MR replaces the use of jQuery in file app/assets/javascripts/projects/star.js with vanilla JavaScript to fix #363000 (closed).

Screenshots or screen recordings

Screen recording of 102db75f in GitPod:

screen_recording

How to set up and validate locally

  1. Start GitLab from this branch
  2. Login to GitLab and navigate to a projects, like http://127.0.0.1:3000/flightjs/Flight from the GitLab Development Kit default seeding
  3. In the project button panel on the upper right, click 'Star'/'Unstar'
  4. The project should toggle between starred and unstarred for your user

MR acceptance checklist

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

Related to #363000 (closed)

Edited by Daniel Schömer

Merge request reports