Migrate to shared alert HAML partial in _commits.html.haml
requested to merge 325369-migrate-to-shared-alert-haml-partial-in-app-views-projects-commits-_commits-html-haml into master
What does this MR do and why?
Update app/views/projects/commits/_commits.html.haml
to leverage the shared alert HAML partial (app/views/shared/_global_alert.html.haml
)
Screenshots or screen recordings
Before | After |
---|---|
How to set up and validate locally
- Quickest way to enable this alert is to update
COMMITS_SAFE_SIZE
inmerge_request_diff.rb
to be a small number (eg. 1) - Visit a projects commits page, eg. http://localhost:3000/flightjs/Flight/-/commits/master
MR acceptance checklist
This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.
-
I have evaluated the MR acceptance checklist for this MR.
Related to #325369 (closed)
Edited by Vamsi Vempati