Copilot commented on code in PR #6280:
URL: https://github.com/apache/texera/pull/6280#discussion_r3548297828
##########
frontend/src/app/dashboard/component/user/list-item/card-item/card-item.component.ts:
##########
@@ -194,7 +194,6 @@ export class CardItemComponent implements OnChanges {
}
initializeEntry() {
- this.coverImageSrc = CardItemComponent.DEFAULT_PREVIEW_IMAGE;
if (this.entry.type === "workflow") {
if (typeof this.entry.id === "number") {
Review Comment:
`initializeEntry()` no longer resets `coverImageSrc` to the default before
branching. When the component input switches from a workflow (with a cover) to
a project/file or to a dataset without a cover, the previous card image can
remain displayed because those branches don't synchronously set `coverImageSrc`.
##########
frontend/src/app/dashboard/type/dashboard-workflow.interface.ts:
##########
@@ -26,4 +26,5 @@ export interface DashboardWorkflow {
projectIDs: number[];
accessLevel: string;
ownerId: number;
+ coverImage: string | null
Review Comment:
Missing trailing semicolon on the new `coverImage` property; this file
consistently terminates interface fields with semicolons, and this may violate
lint/format rules.
--
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]
