ewingjm / development-hub

A continuous integration solution for Power Apps.
MIT License
35 stars 8 forks source link

fix: manual retry needed if master instance paused #51

Closed ewingjm closed 3 years ago

ewingjm commented 3 years ago

Purpose

Fixes #26, fixes #12. Solution merges would occasionally fail with a message stating that the solution cannot be imported because of another solution import in progress. This was due to the fact that it would attempt to import the solution again if the action exceeded the 2-minute timeout.

Approach

Refactoring of the merging flow to not automatically retry the solution import. Instead, we query for the import job whether the solution import action succeeded or timed out.

This pull request also contains some wider refactoring, moving the majority of the code to flow and moving the app registration details from plug-in step configuration to the Environment table.

BREAKING CHANGE: users that update to this version will need to re-enter their app registration details in their environment record(s) (previously within plug-in step secure configuration).

TODOs