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

PowerPlatform integration #927

Merged
merged 200 commits into from
May 14, 2024
Merged

PowerPlatform integration #927

merged 200 commits into from
May 14, 2024

Conversation

freddydk
Copy link
Contributor

@freddydk freddydk commented Feb 9, 2024

Fixes #361

New Settings:

  • PowerPlatformSolutionFolder - containing the name of the folder containing a PowerPlatform Solution (only one)
  • companyId and ppEnvironmentUrl added to deploymentSettings for environments.

New Actions:

  • BuildPowerPlatform - to build a PowerPlatform Solution
  • DeployPowerPlatform - to deploy a PowerPlatform Solution
  • PullPowerPlatformChanges - to pull changes made in PowerPlatform studio into the repository
  • ReadPowerPlatformSettings - to read settings and secrets for PowerPlatform deployment
  • GetArtifactsForDeployment - originally code from deploy.ps1 to retrieve artifacts for releases or builds - now as an action to read apps into a folder.

New Workflows:

  • Pull PowerPlatform Changes
  • Push PowerPlatform Changes

Other changes

  • Getting artifacts for deployment moved from deploy.ps1 to a seperate action
  • Test for specific version of containerhelper moved to avoid many warnings
  • Add scenarios for PowerPlatform
  • Add PowerPlatformSolution artifact to builds
  • Add unpack parameter to DownloadArtifact to unpack after download.

TO:DO

  • Fix failing CI tests
  • Ensure End 2 End test are passing
  • Increment version number in PowerPlatform project (awaits Increment Version Number PR from @mazhelez)
  • Unit Tests
  • End 2 End test
  • Remove PREVIEW prefix from various docs
  • Remove usage of private version of BcContainerHelper
  • Release notes
  • Document new settings

@freddydk freddydk requested review from mazhelez and aholstrup1 May 3, 2024 08:43
Copy link
Collaborator

@mazhelez mazhelez left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks alright.

The checks for some jobs seem unnecessarily complex now, can you double check if all the checks are needed?

Actions/PullPowerPlatformChanges/action.yaml Outdated Show resolved Hide resolved
mazhelez
mazhelez previously approved these changes May 13, 2024
RELEASENOTES.md Show resolved Hide resolved
mazhelez
mazhelez previously approved these changes May 13, 2024
aholstrup1
aholstrup1 previously approved these changes May 13, 2024
@freddydk freddydk dismissed stale reviews from aholstrup1 and mazhelez via b440755 May 13, 2024 11:47
@freddydk freddydk requested review from mazhelez and aholstrup1 May 13, 2024 16:36
@freddydk
Copy link
Contributor Author

Had some last fixes due to end 2 end tests failing - passes now.
Pls approve and I will merge and deploy to preview

@freddydk freddydk merged commit 7a8c4a7 into microsoft:main May 14, 2024
4 checks passed
@freddydk freddydk deleted the PPP branch June 22, 2024 20:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support a PowerPlatform Solution in AL-Go PTE
4 participants