Skip to content

Resolve "Combine Project Repo Buttons to reduce the number of Vue apps"

Andrew Smith requested to merge espadav8/gitlab:439557-combine-project-repo-buttons-to-reduce-the-number-of-vue-apps into master

What does this MR do and why?

Combines a number (4) of Vue apps

  • NotificationsDropdown
  • StarCount
  • ForksButton
  • MoreActionsDropdown

Into a single RepoButtons Vue app

MR acceptance checklist

Please evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.

Screenshots or screen recordings

These are just to show that before they were individual apps, and now they are all contained within a single Vue app.

Before After
image image

How to set up and validate locally

Numbered steps to set up and validate the change are strongly suggested.

  1. Pull down the MR
  2. Visit a project show page, for example http://gdk.test:3000/flightjs/Flight
  3. Check that the panel buttons are still present and look the same
  4. Using the Vue dev tools, check to see if there is a single Vue app for the buttons

Related to #439557 (closed)

Edited by Andrew Smith

Merge request reports