fix: approved by always solution merge creator - Githubissues

ewingjm / development-hub

A continuous integration solution for Power Apps.

MIT License

35 stars 8 forks source link

fix: approved by always solution merge creator #49

Closed ewingjm closed 3 years ago

ewingjm commented 3 years ago

Purpose

Fixes #24, which was due to the fact that we were updating the status reason to 'Approved' and then setting 'Approved By' to 'Modified By'.

Approach

Sets the status to reason to 'Approved' before setting 'Approved By' to 'Modified By'.

TODOs

[x] Automated test coverage for new code
[x] Documentation updated (if required)
[x] Build and tests successful