Merge pull request #394 from justin-luoma/patch-1

SudKul · web-flow · commit ad8955075ad1 · 2022-06-03T18:12:04.000+05:30
fix: prevent posting duplicate records
diff --git a/course-02/exercises/udacity-c2-frontend/src/app/api/api.service.ts b/course-02/exercises/udacity-c2-frontend/src/app/api/api.service.ts
@@ -63,7 +63,7 @@ export class ApiService {
 
     return new Promise ( resolve => {
         this.http.request(req).subscribe((resp) => {
-        if (resp && (<any> resp).status && (<any> resp).status === 200) {
+        if (resp && resp.type === HttpEventType.Response && (<any> resp).status && (<any> resp).status === 200) {
           resolve(this.post(endpoint, payload));
         }
       });

Original file line number	Diff line number	Diff line change
`@@ -63,7 +63,7 @@ export class ApiService {`
`63`	`63`
`64`	`64`	`return new Promise ( resolve => {`
`65`	`65`	`this.http.request(req).subscribe((resp) => {`
`66`		`- if (resp && (<any> resp).status && (<any> resp).status === 200) {`
	`66`	`+ if (resp && resp.type === HttpEventType.Response && (<any> resp).status && (<any> resp).status === 200) {`
`67`	`67`	`resolve(this.post(endpoint, payload));`
`68`	`68`	`}`
`69`	`69`	`});`