Merge pull request #368 from forcedotcom/sm/no-tracking-when-sdr-even…

…ts-are-not-successful fix: no tracking on failed deploys
forcedotcom · Apr 13, 2023 · d61489f · d61489f
2 parents 3b4bcb0 + 4256d01
commit d61489f
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/src/sourceTracking.ts b/src/sourceTracking.ts
@@ -656,13 +656,13 @@ export class SourceTracking extends AsyncCreatable {
       // yes, the post hooks really have different payloads!
       lifecycle.on('scopedPostDeploy', async (e: ScopedPostDeploy) => {
         this.logger.debug('received scopedPostDeploy event');
-        if (e.orgId === this.orgId) {
+        if (e.orgId === this.orgId && e.deployResult.response.success) {
           await this.updateTrackingFromDeploy(e.deployResult);
         }
       });
       lifecycle.on('scopedPostRetrieve', async (e: ScopedPostRetrieve) => {
         this.logger.debug('received scopedPostRetrieve event');
-        if (e.orgId === this.orgId) {
+        if (e.orgId === this.orgId && e.retrieveResult.response.success) {
           await this.updateTrackingFromRetrieve(e.retrieveResult);
         }
       });