Skip to content
This repository has been archived by the owner on Mar 11, 2022. It is now read-only.

CrossDomainEnabled + CrossDomainEnabledFast can be merged or inherit each other #480

Open
wsdt opened this issue Mar 10, 2022 · 0 comments
Open

CrossDomainEnabled + CrossDomainEnabledFast can be merged or inherit each other #480

wsdt opened this issue Mar 10, 2022 · 0 comments
Assignees
@souradeep-das
Labels
question Further information is requested

Comments

@wsdt
Copy link
Contributor

wsdt commented Mar 10, 2022

At first sight it looks like we could easily merge both contracts or at least one extending the other.

CrossDomainEnabledFast <-> CrossDomainEnabled

Definitely low prio, but optimizations like that reduce boilerplate/duplicated code -> easier maintenance, etc.
@wsdt wsdt added the question Further information is requested label Mar 10, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@souradeep-das souradeep-das

Labels
question Further information is requested
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@souradeep-das @wsdt