Fix Web IDE renaming empty content
What does this MR do and why?
A small(temporary) fix for: #296912 (closed). In this fix, we remove a current ternary from createCommitPayload
which would often null
the content of a staged file from our new commit.
THIS IS NOT A HOLISTIC FIX.
The holistic fix would be to discover why sometimes the files changed are not toggled with the chnaged: true
Boolean inside the rootState.stagedFiles
array.
Screenshots or screen recordings
How to set up and validate locally
- Log in to GitLab as an admin:
root
/5iveL!fe
- Navigate to any seeded project i.e.
http://127.0.0.1:3000/flightjs/Flight
- Click
Web IDE
to open the Web IDE. - Rename, edit and update a large selection of files.
- Click commit
- Open the
Changes
tab to see the correct changes reflected(Renamed, moved, staged, edited, deleted etc).
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 #296912 (closed)
Edited by David O'Regan