Skip to content
New issue

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

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

Already on GitHub? # to your account

feat: improve signal kinds for ranges and integrals #2652

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

hamed-musallam
Copy link
Member

@hamed-musallam hamed-musallam commented Aug 30, 2023

I think it could be better to have a special react context for the computed values instead of having a custom hook for each one.

@hamed-musallam hamed-musallam linked an issue Aug 30, 2023 that may be closed by this pull request
@cloudflare-workers-and-pages
Copy link

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: ae6ffdf
Status: ✅  Deploy successful!
Preview URL: https://440b5650.nmrium.pages.dev
Branch Preview URL: https://improve-kind-in-ranges-and-i.nmrium.pages.dev

View logs

@hamed-musallam hamed-musallam linked an issue Aug 30, 2023 that may be closed by this pull request
@codecov
Copy link

codecov bot commented Aug 30, 2023

Codecov Report

Patch coverage: 100.00% and project coverage change: -0.46% ⚠️

Comparison is base (a884085) 53.40% compared to head (ae6ffdf) 52.95%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2652      +/-   ##
==========================================
- Coverage   53.40%   52.95%   -0.46%     
==========================================
  Files          52       52              
  Lines        2584     2559      -25     
  Branches       91       91              
==========================================
- Hits         1380     1355      -25     
  Misses       1202     1202              
  Partials        2        2              
Files Changed Coverage Δ
src/data/constants/SignalsKinds.ts 100.00% <100.00%> (ø)

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@lpatiny
Copy link
Member

lpatiny commented Sep 5, 2023

When renaming ranges become undefined.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Improve the menu 'Kind' Kind in ranges and integrals
2 participants