Brijesh-Thakkar commented on code in PR #4076:
URL: https://github.com/apache/streampipes/pull/4076#discussion_r2644924871
##########
ui/src/app/assets/components/asset-overview/asset-overview.component.ts:
##########
@@ -163,6 +163,8 @@ export class SpAssetOverviewComponent implements OnInit {
dialogRef.afterClosed().subscribe(ev => {
if (ev) {
+ this.loadAssets();
+ this.assetBrowserService.reloadAssetData();
Review Comment:
Good point, thanks for highlighting this.
I’ve updated the implementation to consistently use `loadAssetData()` across
the UI instead of introducing a second reload API. The previously added
`reloadAssetData()` wrapper was removed to avoid duplication and keep the asset
reload behavior consistent with existing usage (e.g., asset deletion and save).
This keeps the fix minimal and improves maintainability without expanding
the scope of the PR.
--
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]
