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.

Sign up for GitHub

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

Jump to bottom

Get rid of RxJS #32

Open
2 tasks done
DeltaZN opened this issue Sep 14, 2023 · 1 comment
Open
2 tasks done

Get rid of RxJS #32

DeltaZN opened this issue Sep 14, 2023 · 1 comment
Labels
enhancement: need to review New feature / improvement request. Need to review

Comments

@DeltaZN
Copy link
Contributor

DeltaZN commented Sep 14, 2023 •
edited
Loading

Duplicates

  • I have searched the proposed features

Latest version

  • I have tested the latest version

Suggestion ⭐

RxJS bundle is quite big, moreover, some projects might not use it, however, they'll have to include this dependency in their bundle anyway.
I suggest writing a lightweight implementation which will follow this proposal.
Also, we need to make sure that it's compatible with rxjs, since there are existing projects which uses our current implementaion with rxjs

Motivation 🔦

Make dxcharts-lite really lite
@DeltaZN DeltaZN added the enhancement: need to review New feature / improvement request. Need to review label Sep 14, 2023
@dxpm
Copy link
Contributor

dxpm commented Sep 14, 2023 •
edited
Loading

Perhaps we should consider using something like zen-observable or maybe fork of it could be good start.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement: need to review New feature / improvement request. Need to review
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@DeltaZN @dxpm