Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(core): generic repository events #842

Conversation

Przytua
Copy link
Contributor

@Przytua Przytua commented Jun 9, 2022

Closes #723

Added generic RecordSavedEvent, RecordUpdatedEvent, and RecordDeletedEvent emitted from Repository.

@Przytua Przytua requested a review from a team as a code owner June 9, 2022 15:03
@Przytua Przytua mentioned this pull request Jun 9, 2022
Copy link
Contributor

@TimoGlastra TimoGlastra left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is awesome @Przytua, thanks!

It seems CI is failing and there's some eslint warnings but code looks good otherwise

RecordDeleted = 'RecordDeleted',
}

export interface RecordSavedEvent<T extends BaseRecord<any, any, any>> extends BaseEvent {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can you disable the eslint warnings for this line?

}
}

export interface RecordUpdatedEvent<T extends BaseRecord<any, any, any>> extends BaseEvent {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

same here

}
}

export interface RecordDeletedEvent<T extends BaseRecord<any, any, any>> extends BaseEvent {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

same here

@Przytua Przytua force-pushed the feature/generic-repository-events branch from a4259f7 to 180976a Compare June 9, 2022 16:53
Signed-off-by: Łukasz Przytuła <[email protected]>
@Przytua Przytua force-pushed the feature/generic-repository-events branch from 180976a to c48ec8d Compare June 9, 2022 17:15
@Przytua
Copy link
Contributor Author

Przytua commented Jun 9, 2022

@TimoGlastra fixed

@TimoGlastra TimoGlastra changed the title feat(core): Generic Repository events feat(core): generic repository events Jun 9, 2022
@TimoGlastra TimoGlastra merged commit 74dd289 into openwallet-foundation:main Jun 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Generic Repository Events
2 participants