Skip to content

Conversation

@devkeydet
Copy link
Contributor

Closes #71

@marcelbf
Copy link
Collaborator

I could not make it work in my environment.

pac code push works and replaces the appId... but app doesn't run...
image

npm run dev works, but will not load the data:
image

tested in 2 different environments - both First region.

can you confirm we are using same pac cli version?

image

@marcelbf marcelbf self-requested a review August 12, 2025 22:02
@marcelbf marcelbf marked this pull request as draft August 19, 2025 22:46
@bpenno-altus
Copy link

bpenno-altus commented Aug 27, 2025

I have the same experience running it in my local dev environment using 'npm run dev' - it fails to load the data.

On pushing to my environment, I was also receiving an App timed out issue. I resolved it by removing the following code segment from the vite.config.ts file:

    // Strategic vendor chunking for better long-term caching & smaller initial bundle.
    manualChunks(id) {
      if (id.includes('node_modules')) {
        // Order matters: check more specific packages first to avoid them being captured by generic 'react' substring.
        if (id.includes('@fluentui')) return 'fluentui';
        if (id.includes('@pa-client/power-code-sdk')) return 'power-sdk';
        if (/react-dom|node_modules\\react\\|node_modules\/react\//.test(id)) return 'react';
      }

This was based on a suggestion made elsewhere in this repo.

Using:
Microsoft PowerPlatform CLI
Version: 1.47.1+gc5a7855 (.NET Framework 4.8.9310.0)

@marcelbf
Copy link
Collaborator

@bpenno-altus thanks for testing.
We identified the issue. Should be fixed soon in a new CLI release, then we can merge.

@Cliveo
Copy link

Cliveo commented Oct 3, 2025

I got a critical warning from npm for @pa-client/power-code-sdk, is this a mistake?

GHSA-484g-hww8-cwwm
https://www.npmjs.com/package/@pa-client/power-code-sdk

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.

Add a Dataverse sample

5 participants