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

Ruth/release/v44.2.0 #666

Merged
merged 7 commits into from
Oct 9, 2018
Merged

Conversation

ruthemmanuelle
Copy link
Contributor

What does this PR do?

Updates docs to reflect changes in v44 of Salesforce Extensions for VS Code.

What issues does this PR fix or reference?

@W-5444466, W-5220476@
#123

- You can use this debugger only in your orgs. ISV customer debugging is unavailable in Apex Replay Debugger. To debug customers’ orgs, use [ISV Customer Debugger](https://marketplace.visualstudio.com/items?itemName=salesforce.salesforcedx-vscode-apex-debugger).
- You can replay only one debug log at a time. This limitation can make it difficult to debug asynchronous Apex, which produces multiple debug logs.
- Be sure to start a session soon after uploading your checkpoints, because checkpoints expire after 30 minutes.
- Be sure to debug your code soon after starting the session, because heap dumps expire about a day after you generate them.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Everything in this list below the line about heap dump expiration is unchanged. It shows as changed because I changed the asterisks to hyphens.

@codecov
Copy link

codecov bot commented Oct 9, 2018

Codecov Report

Merging #666 into release/v44.2.0 will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@               Coverage Diff                @@
##           release/v44.2.0     #666   +/-   ##
================================================
  Coverage            74.87%   74.87%           
================================================
  Files                  158      158           
  Lines                 6431     6431           
  Branches              1009     1009           
================================================
  Hits                  4815     4815           
  Misses                1358     1358           
  Partials               258      258

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 34d52a8...3f6a583. Read the comment docs.

README.md Outdated
@@ -17,7 +17,7 @@ Currently, we have the following extensions:
- [salesforcedx-vscode-apex](https://marketplace.visualstudio.com/items?itemName=salesforce.salesforcedx-vscode-apex)
This extension uses the Apex Language Server to provide features such as syntax highlighting and code completion.
- [salesforcedx-vscode-apex-debugger](https://marketplace.visualstudio.com/items?itemName=salesforce.salesforcedx-vscode-apex-debugger)
This extension enables VS Code to use the real-time Apex Debugger with your scratch orgs.
This extension enables VS Code to use the real-time Apex Debugger with your scratch orgs or to use ISV Customer Debugger.
Copy link
Contributor

Choose a reason for hiding this comment

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

You use "and" instead of "or" in example just below. Can they use both at once or either?

Copy link
Contributor

Choose a reason for hiding this comment

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

See my next comment.


- Or, to convert an existing breakpoint into a checkpoint, right-click the breakpoint, and select **Edit Breakpoint** | **Expression**. Set the expression to `Checkpoint`.

1. To upload your checkpoints to your org so that heap dump information can be collected, open the command palette, and run **SFDX: Update Checkpoints in Org**.
Copy link
Contributor

Choose a reason for hiding this comment

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

editing nit: "To upload your checkpoints to your org to collect heap dump info, open the..."

@@ -69,7 +69,7 @@ The Apex outline view shows the structure of the Apex class or trigger that’s
## Enable Code Smartness for SObjects
To activate this extension’s code smartness features for standard and custom objects and their fields, including for custom fields on standard objects, press Cmd+Shift+P (macOS) or Ctrl+Shift+P (Windows or Linux), and then select **SFDX: Refresh SObject Definitions** from the command palette.

When you refresh your sObject definitions, VS Code uses your default scratch org to generate faux Apex classes. These faux classes represent the standard and custom objects that the admin user of your default scratch org has access to. The classes are stored in a hidden directory on your local workstation. Don’t edit the faux classes! They are deleted and regenerated each time that you refresh your sObject definitions. To modify your sObjects, either modify the objects’ `.object-meta.xml` and `.field-meta.xml` files (and then run **SFDX: Push Source to Default Scratch Org**), or make changes declaratively in your scratch org (and then run **SFDX: Pull Source from Default Scratch Org**). The scratch org’s admin user doesn’t automatically gain access to new custom objects, so be sure to assign new permissions to the user as necessary. To assign permissions from the command line, run `sfdx force:user:permset:assign -n YourPermSetName`.
When you refresh your sObject definitions, VS Code uses your default org to generate faux Apex classes. These faux classes represent the standard and custom objects that the admin user of your default scratch org, or the logged-in user of your sandbox or DE org, has access to. The classes are stored in a hidden directory on your local workstation. Don’t edit the faux classes! They are deleted and regenerated each time that you refresh your sObject definitions. To modify your sObjects, either modify the objects’ `.object-meta.xml` and `.field-meta.xml` files (and then run **SFDX: Push Source to Default Scratch Org** or **SFDX: Deploy Source to Org**), or make changes declaratively in your org (and then run **SFDX: Pull Source from Default Scratch Org** or **SFDX: Retrieve Source from Org**). Your user doesn’t automatically gain access to new custom objects, so be sure to assign new permissions to the user as necessary. To assign permissions from the command line, run `sfdx force:user:permset:assign -n YourPermSetName`.
Copy link
Contributor

Choose a reason for hiding this comment

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

Because of the addition of sandbox/DE, this sentence becomes less readable. So I suggest linking it back together:
These faux classes represent the standard and custom objects that the admin user has access to for your default scratch org, or the logged-in user of your sandbox or DE org.

* **[Visual Studio Code](https://code.visualstudio.com/download) v1.23 or later**

## Set Up a Default Scratch Org
To access the Visual Studio Code command palette, press Cmd+Shift+P (macOS) or Ctrl+Shift+P (Windows or Linux). To create a scratch org and set it as your default org for development, run **SFDX: Authorize a Dev Hub** and then **SFDX: Create a Default Scratch Org**. Then, to push the source in our project to the scratch org, run **SFDX: Push Source to Default Scratch Org**. To open the org in your browser, run **SFDX: Open Default Org**. After you make changes in the Salesforce user interface, to pull those changes to your local project, run **SFDX: Pull Source from Default Scratch Org**.
Copy link
Contributor

@ekapner ekapner Oct 9, 2018

Choose a reason for hiding this comment

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

I suspect you mean "your"
Then, to push the source in our project to the scratch org,...

1. To create a scratch org, you need a Dev Hub. To set up your production org as a Dev Hub, see [Enable Dev Hub in Your Org](https://developer.salesforce.com/docs/atlas.en-us.sfdx_setup.meta/sfdx_setup/sfdx_setup_enable_devhub.htm) in the _Salesforce DX Setup Guide_.
1. To authorize your Dev Hub, open VS Code’s command palette (Cmd+Shift+P on macOS, or Ctrl+Shift+P on Windows or Linux) and run **SFDX: Authorize a Dev Hub**.
1. To create a default scratch org, run **SFDX: Create a Default Scratch Org**. Then, run **SFDX: Push Source to Default Scratch Org**. For more information, see [Scratch Orgs](https://developer.salesforce.com/docs/atlas.en-us.sfdx_dev.meta/sfdx_dev/sfdx_dev_scratch_orgs.htm) in the _Salesforce DX Developer Guide_.
* **[Visual Studio Code](https://code.visualstudio.com/download) v1.23 or later**
* **[Visual Studio Code](https://code.visualstudio.com/download) v1.23 or later**
Copy link
Contributor

Choose a reason for hiding this comment

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

We should change this to be 1.26 or later

Open your Salesforce DX project in a directory that contains an `sfdx-project.json` file. Otherwise, some features don’t work.
If you don't already have a Salesforce DX project, create one with the **SFDX: Create Project** command (for development against scratch orgs) or the **SFDX: Create Project with Manifest** command (for development against sandboxes or DE orgs). Or, see [create a Salesforce DX project](https://developer.salesforce.com/docs/atlas.en-us.sfdx_dev.meta/sfdx_dev/sfdx_dev_workspace_setup.htm) for information about setting up a project using Salesforce CLI.
* **[Visual Studio Code](https://code.visualstudio.com/download) v1.23 or later**
Copy link
Contributor

Choose a reason for hiding this comment

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

Same as above, change to 1.26 or later

@@ -1,5 +1,5 @@
# Apex Debugger for Visual Studio Code
This extension enables VS Code to use the real-time Apex Debugger with your scratch orgs.
This extension enables VS Code to use the real-time Apex Debugger with your scratch orgs and to use ISV Customer Debugger.
Copy link
Contributor

Choose a reason for hiding this comment

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

After reading completely thru, sounds like you may want to add "...Apex Debugger with your scratch orgs, or to use ISV Customer Debugger for sandboxes or DE orgs." Something like that.

@@ -21,7 +23,7 @@ To use Salesforce Extensions for VS Code, install all the extensions in this ext
* [salesforcedx-vscode-apex](https://marketplace.visualstudio.com/items?itemName=salesforce.salesforcedx-vscode-apex)
This extension uses the Apex Language Server to provide features such as syntax highlighting and code completion.
* [salesforcedx-vscode-apex-debugger](https://marketplace.visualstudio.com/items?itemName=salesforce.salesforcedx-vscode-apex-debugger)
This extension enables VS Code to use the real-time Apex Debugger with your scratch orgs.
This extension enables VS Code to use the real-time Apex Debugger with your scratch orgs or to use ISV Customer Debugger.
Copy link
Contributor

Choose a reason for hiding this comment

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

Same comment as above - but now I'm going to ask if ISV cust debugger can be used with any org?

Copy link
Contributor

Choose a reason for hiding this comment

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

Just would say if you want to say here ...for ISV Customer Debugger for your subscribers' orgs. But I will defer to your discretion.

Copy link
Contributor

@ekapner ekapner left a comment

Choose a reason for hiding this comment

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

Some feedback and questions to help clarify a couple things.

@lcampos
Copy link
Contributor

lcampos commented Oct 9, 2018

We also need to update the wiki (https://github.com/forcedotcom/salesforcedx-vscode/wiki/Develop-Against-Any-Org-in-Visual-Studio-Code) as part of v44. @ruthemmanuelle is this something that should be done as part of a different review ?

@W-5444466, W-5220476@
@W-5444466, W-5220476@
@ruthemmanuelle
Copy link
Contributor Author

ruthemmanuelle commented Oct 9, 2018

@ntotten - See @lcampos 's comment about the wiki article. Are you going to update that (or delete it)? (We can't update it as part of this PR because you can't do PRs for wiki articles.)

@W-5444466, W-5220476@
Copy link
Contributor

@JimSuplizio JimSuplizio left a comment

Choose a reason for hiding this comment

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

I'm good with the updates. I know you're still making changes for other feedback but I'm happy with the checkpoints additions but I'm going to approve my piece. Thank you Ruth!

@lcampos
Copy link
Contributor

lcampos commented Oct 9, 2018

@ntotten - See @lcampos 's comment about the wiki article. Are you going to update that (or delete it)? (We can't update it as part of this PR because you can't do PRs for wiki articles.)

BTW, @ntotten is out until next week so I'll take care of updating the wiki article as part of the release.

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.

None yet

4 participants