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

Issue on problem panel not get clear after successful deployment #4904

Closed
vikaskumar171020 opened this issue Jun 3, 2023 · 32 comments · Fixed by #4962
Closed

Issue on problem panel not get clear after successful deployment #4904

vikaskumar171020 opened this issue Jun 3, 2023 · 32 comments · Fixed by #4962

Comments

@vikaskumar171020
Copy link

vikaskumar171020 commented Jun 3, 2023

Hi i the latest release there is issue i found sometime after deploying code it show error for unsuccessfully code deployment and the error show even after the code deployed successfully which is not clear the problem in the file.

FOR BUGS RELATED TO THE SALEFORCE CLI, please use this repository: https://github.com/forcedotcom/cli

Summary

Short summary of what is going on or to provide context.

Steps To Reproduce:

  1. This is step 1.
  2. This is step 2. All steps should start with '1.'

Expected result

problem need to clear after successful deployment

Actual result

problem still appear after successful deployment

Salesforce Extension Version in VS Code:
Salesforce CLI Integration version "v58.0.1"

SFDX CLI Version:
sfdx-cli/7.203.6 win32-x64 node-v18.15.0

image

@yuki18886
Copy link

downgrading following extensions from 58.0.1 to 57.15.0 worked for me:

Apex
Apex Replay Debugger
Aura Components
Lightning Web Components
Salesforce CLI Integration
Salesforce Extension Pack

how you can choose a previous version:
#3901 (comment)

@CristiCanizales
Copy link
Contributor

Hi @vikaskumar171020 ! Could you please send screenshots of the error you're referring to?

@vikaskumar171020
Copy link
Author

yeah already did that but due to auto update feature all extension gets updated which cause this issue come over and over again

@yuki18886
Copy link

yuki18886 commented Jun 6, 2023

@vikaskumar171020
You may consider turning off auto-update settings, if you are in a situation where you can manage your personal settings.

see also:
Extension auto-update
Update an extension manually

@Fosnez
Copy link

Fosnez commented Jun 7, 2023

Yup, also affected by this.

@akalatksy
Copy link

akalatksy commented Jun 7, 2023

Also affected by this.
Downgrade of extensions mentioned by @yuki18886 helped. Thanks @yuki18886 ! 👍

@vikaskumar171020
Copy link
Author

yeah thanks that work for me @yuki18886

@bobalicious
Copy link

bobalicious commented Jun 8, 2023

Any update on this? This is a major obstacle for development - it gets in the way constantly - big downgrade in functionality.

Note, issue it not just on successful deployment, but on all subsequent deployments whether successful or not. You end up with highlights for all the previous compilation errors that have been on the file.

@cardinjz
Copy link

cardinjz commented Jun 8, 2023

Somehow completely missed that this issue had been created already, but wanted to point out that it looks like the same problem came up with #2608 in case it helped with review.

@AndrewStopchenko-SO
Copy link
Contributor

AndrewStopchenko-SO commented Jun 9, 2023

downgrading following extensions from 58.0.1 to 57.15.0 worked for me:

Apex Apex Replay Debugger Aura Components Lightning Web Components Salesforce CLI Integration Salesforce Extension Pack

how you can choose a previous version: #3901 (comment)

Please note that it could be enough to downgrade only the "Salesforce CLI Integration" extension to v57.15.0. Thanks!

@vikaskumar171020
Copy link
Author

yes i figured out that as well but it's important to resolve the issue so extension can maintain according to sfdx cli
thanks for all your support

@clint-rosetree
Copy link

Also affected by this bug. It's a real time and focus waster to double-check for non-existent problems while coding.

@neilhohmann
Copy link

Rolling back the Salesforce CLI Integration from v58.2 to v57.15 seems to have worked for me. I also changed Update Mode (under Settings) to 'None'.

@willvoyle
Copy link

Workaround steps are below:

  1. Open VS Code
  2. Extensions Tab
  3. Search for 'Salesforce CLI Integration'
  4. Click the down arrow next to 'Uninstall'
  5. Click 'Install Another Version...'
  6. Select '57.15'
  7. Reload VS Code

@PedroPTavaresJr
Copy link

Also got affected by this.
downgrading the Salesforce CLI Integration extension, to 57.15.0 works like a charm.
as of today, 206/26/2023, we are on 58.3.1 - 5 updates later, no fix.

@ccalvomartinez
Copy link

Also affected by this.
Please, we need this to be resolved as soon as possible.

@evakwright
Copy link

Also affected. I found it easier for me to ctrl-shift-p reload window than downgrading the CLI.

@CristiCanizales
Copy link
Contributor

Hello everyone! We're definitely aware of this and are sorry for the delay. This is next in the top of my todo's list. I'll let you all know once the fix is in.

@ccalvomartinez
Copy link

Thank you very much, @CristiCanizales!

@CristiCanizales
Copy link
Contributor

Hello everyone!
I just wanted to let you know that we already have the fix for this (⬆️) that will likely go out in our next release, next week ~Wednesday. Thank you all for your patience. I'll announce here when it's all done ✅

@CristiCanizales
Copy link
Contributor

Hello everyone! v58.6.2 of the Salesforce Extensions for VS Code has been published and includes the fix for this issue 🚀 Thank you all for your patience!

@bobalicious
Copy link

Have upgraded to this version and the issue still exists.

I notice on the pull request there's the comment:
"Changes have been made in DeployExecutor and jest tests is a heavy lift for the code change at this point in time. So we will cover the test as a separate W-13736554"

I understand the desire to get changes out quickly, but it does feel like the fix would have benefitted from having the unit tests written. That may have avoided the release being made with the fix not working.

@AndrewStopchenko-SO
Copy link
Contributor

AndrewStopchenko-SO commented Jul 13, 2023

Hi, I have just upgraded to v58.6.2 and tried two ways of deploying broken/working Apex code, with results below:

  1. When using the "SFDX: Deploy This Source To Org" command, the issue seems to be fixed:
    1.1. attempts to deploy broken Apex code clears the "Problems" panel's items on each re-deployment, and adds new error item(s), as expected.
    1.2. thus, when successfully deploying working Apex code, the "Problems" panel gets cleared out, as expected.

  2. When using the "SFDX: Push Source to Default Org" command, the issue persists:
    2.1. pushing the broken Apex code keeps adding new errors to the "Problems" panel; a new error entry is added for each unsuccessful source push attempt;
    2.2. pushing the working Apex code does not clear errors added by unsuccessful source push attempts above.
    2.3. trying to run the "SFDX: Deloy This Source To Org" command with working Apex code, code is being deployed successfully, but the problem items added by the push source command (2.1.) are not removed from the "Problems" panel.

So, it looks like the "SFDX: Deploy This Source To Org" command is fixed, but the "SFDX: Push Source to Default Org" command still has the issue.

@CristiCanizales
Copy link
Contributor

Hello @AndrewStopchenko-SO, yes, we just found out about the same issue with push yesterday in the afternoon while doing the testing for the release, and already have a PR open for that, but we need to work on the tests first.

@bobalicious
Copy link

Can this ticket be re-opened then please

@CristiCanizales
Copy link
Contributor

Hi @bobalicious, I wanted to ask if your case was the 2nd way @AndrewStopchenko-SO mentioned in his comment, that the "push errors" are kept in the problems tab

@bobalicious
Copy link

Yes, it is.

@CristiCanizales
Copy link
Contributor

Thanks for your reply @bobalicious. What I'd suggest then as a workaround while the fix for that other issue goes out, to avoid having to reload the window every time you have errors that should not be there, is to stick to the 1st way mentioned in the previous comment, like prefer deploy from the beginning, as that flow is working ok.

@bobalicious
Copy link

Thanks, but I don't need help with a workaround - I have that - lock the version to an earlier one (FYI - using deploy instead of push is not an acceptable work-around because it breaks my workflow too much)

What I need is a way of finding out when I no longer need to use the workaround - previously I had been tracking this bug report. Can you please re-open this issue or direct me to the one you are using to track your fix.

@CristiCanizales
Copy link
Contributor

@bobalicious I'm using this: #4942. It is connected to the PR I have open with the fix, but it needs the tests before going in, which will go along with the tests for this fix.

@bobalicious
Copy link

Thanks, will track that one

@vikaskumar171020
Copy link
Author

the issue is resolved in deploy code however it still exist in source push command

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.