ocket8888 commented on code in PR #7532:
URL: https://github.com/apache/trafficcontrol/pull/7532#discussion_r1205777249
##########
experimental/traffic-portal/src/app/core/profiles/profile-table/profile-table.component.ts:
##########
@@ -170,6 +184,37 @@ export class ProfileTableComponent implements OnInit {
}
});
break;
+ case "export-profile":
+ const response = await
this.exportService.exportProfile(data.id);
+ Reflect.deleteProperty(response, "alerts");
Review Comment:
I suppose that's fine, although note that the API will ignore it on
re-upload via Profile import, so it doesn't matter if it's in there or not,
functionally. But if you really wanna do that, you should just use a regular
`delete` statement. `Reflect` is meant for proxying object-internal methods,
and it's much more complex than what's needed to simply remove a property from
a regular object.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
