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

fix: Org browser ignores sfdx project.json api version #4891

Merged

Conversation

jeffb-sfdc
Copy link
Contributor

@jeffb-sfdc jeffb-sfdc commented May 26, 2023

What does this PR do?

What issues does this PR fix or reference?

#3663, @W-10273533@

Functionality Before

Deploying and retrieving was not adhering to the API rules defined at https://developer.salesforce.com/docs/atlas.en-us.sfdx_setup.meta/sfdx_setup/sfdx_setup_apiversion.htm

Functionality After

Deploy and retrieve now use setApiVersion() (in packages/salesforcedx-vscode-core/src/services/sdr/componentSetUtils.ts) to define which API to use.

NOTE: This is the exact PR as #4801 (and no changes between the two). PR #4801 was merged but needed to be backed out.

@jeffb-sfdc jeffb-sfdc requested a review from a team as a code owner May 26, 2023 22:57
@jeffb-sfdc
Copy link
Contributor Author

This PR is the same as #4801. #4801 was approved, so this PR just needs to be QA'd and then merged.

@jeffb-sfdc jeffb-sfdc changed the title Org browser ignores sfdx project.json api version draft Org browser ignores sfdx project.json api version Jun 1, 2023
@RitamAgrawal
Copy link
Member

QA notes
✅ Version in manifest file takes precedence over sourceApiVersion in sfdx-project.json while

✔️ retrieving source in manifest from org
✔️ deploying source in manifest to Org

✅ sourceApiVersion specified in sfdx-project.json takes precedence over local config while

✔️ retrieving through context menu.
✔️ retrieving through Org Browser
✔️ deploy

@jeffb-sfdc jeffb-sfdc merged commit 3023b5c into develop Jun 6, 2023
@jeffb-sfdc jeffb-sfdc deleted the jb/org-browser-ignores-sfdx-project.json-api-version-draft branch June 6, 2023 23:02
@jeffb-sfdc jeffb-sfdc changed the title Org browser ignores sfdx project.json api version fix: Org browser ignores sfdx project.json api version Jun 13, 2023
@jeffb-sfdc jeffb-sfdc changed the title fix: Org browser ignores sfdx project.json api version feat: Org browser ignores sfdx project.json api version Jun 13, 2023
@jeffb-sfdc jeffb-sfdc changed the title feat: Org browser ignores sfdx project.json api version fix: Org browser ignores sfdx project.json api version Jun 13, 2023
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.

2 participants