Pajamas migrations
What does this MR do and why?
Diff fork cancel button
Screenshots or screen recordings
Before | After |
---|---|
How to set up and validate locally
I couldn't work out how to get this to display. In app/assets/javascripts/diffs/components/diff_file.vue
- delete L404
- remove
v-else
from L423
- Force the button to display following the instructions above
- Go to an MR with file changes
- Go to the changes tab
Closes #428141 (closed)
Diff truncated error retry button
Screenshots or screen recordings
Before | After |
---|---|
How to set up and validate locally
I couldn't work out how to get this to display. In app/assets/javascripts/notes/components/diff_with_note.vue
- delete
v-if="error"
from L129 - delete L135–L139
- Force the button to display following the instructions above
- Go to an MR with text file changes
- Add a comment to the file
- See changes in MR Activity list
Closes #428150 (closed)
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.
Edited by Dan MH