microsoft / microsoft/powerplatform-vscode

pac paportal download does not set previously populated fields to null

Open
#681 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

paportal
Dominant language
TypeScript
Stars
277
Forks
60
Avg merge
1d 1h
Merged PRs (30d)
14

Description

This issue is occurring when running CLI Version: 1.26.5+g670cdf9 via VS Code.

When pac paportal downloading a web page that used to have an adx_webform set but is now null, the key value pair is removed from the yaml file, so when deployed to other environments using pac paportal upload, the value in those environments is not cleared. A workaround we have been doing is when clearing a value from a web page and downloading the yaml, readding the key value pair and setting it to null. These fields not being cleared results in an error when viewing these pages as two copies of the same webform are trying to be rendered. This is because the Basic Form field in the environment had not been cleared, but the new code to render the basic form using liquid was also present.

A solution to this would be to keep the key value pairs in the yaml but set them to null when null in the source environment.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start at the implementation of the pac paportal download and upload commands, tracing how source values are serialized into and read from YAML. Reproduce a page field changing from a populated adx_webform to null, then verify that the downloaded YAML preserves the key with a null value and that upload clears the destination field.

Written by the indexing model from the issue text.

Assessment

Tech stack
typescript
Domain
cli
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
42/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.