altinn-studio icon indicating copy to clipboard operation
altinn-studio copied to clipboard

10837 Review changed files in Altinn Studio

Open standeren opened this issue 7 months ago • 1 comments

Description

  • Remove hardcoded 1 in notification for shareChanges button
  • Remove conditional Mergeconflict text in buttons for shareing and fetching changes
  • Add button to review changes in shareChangesPopup if there are local changes
  • Implement a modal to review fileNames and connected fileStatus
Skjermbilde 2024-07-04 kl  14 17 07 Skjermbilde 2024-07-04 kl  14 16 39

https://github.com/Altinn/altinn-studio/assets/71079896/a8406e2e-a946-414d-8729-662a2321fbbd

  • Also added some padding to the LocalChangesModal Skjermbilde 2024-07-04 kl  14 07 45

Out of scope in this PR: implement support to review actual diff for each file. Should be done in a different PR

Related Issue(s)

  • #{issue number}

Verification

  • [x] Your code builds clean without any errors or warnings
  • [x] Manual testing done (required)
  • [x] Relevant automated test added (if you find this hard, leave it and we'll help out)

standeren avatar Jul 04 '24 11:07 standeren