Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

refactor: migrate to nuke/power apps cli/sdk-style projects #46

Merged
merged 34 commits into from
Nov 29, 2020

Conversation

ewingjm
Copy link
Owner

@ewingjm ewingjm commented Nov 27, 2020

Purpose

  • Cake intellisense was broken due to a mismatch in framework dependencies with one of the Cake.Xrm add-ins. Cake also requires an add-in for intellisense
  • The Cake.Xrm add-ins are not yet using OAuth connection strings and can't connect to newly created environments due to WS-Trust being deprecated
  • The Cake.Xrm add-ins embed Power Apps SDK assemblies in their packages. Therefore, the Cake.Xrm packages must be updated in line with updates to the SDK packages.
  • The build tasks required user credentials to be stored in plaintext environment variables
  • The solution projects were not created using the Power Apps CLI and using the CLI is the easiest way to integrate custom controls and their build tasks
  • None of the projects were SDK-style, which is much less complex and allows the .csproj files to be edited in VS without unloading first

Approach

  • Migrated from Cake to Nuke, which has more lightweight support for CLI tools and does not rely on third-party extensions for intellisense
  • Integrated current solutions onto Power Apps CLI style projects using pac solution clone
  • Updated build tasks to use Power Apps CLI where possible, eliminating the need to store credentials in environment variables (authentication is handled through pac)
  • Migrated to SDK-style projects

TODOs

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

@ewingjm ewingjm added documentation Improvements or additions to documentation enhancement New feature or request labels Nov 29, 2020
@ewingjm ewingjm marked this pull request as ready for review November 29, 2020 08:26
@ewingjm ewingjm merged commit f15a650 into master Nov 29, 2020
@ewingjm ewingjm deleted the refactor/migrate-to-power-apps-cli branch November 29, 2020 09:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant