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

refactor: range editing modal #3417

Merged
merged 1 commit into from
Mar 24, 2025

Conversation

hamed-musallam
Copy link
Member

No description provided.

@hamed-musallam hamed-musallam linked an issue Mar 18, 2025 that may be closed by this pull request
@hamed-musallam hamed-musallam marked this pull request as draft March 18, 2025 17:07
@hamed-musallam hamed-musallam force-pushed the fix-adding-peak-crash-advance-mode branch from 8008493 to 910859a Compare March 18, 2025 17:12
Copy link

cloudflare-workers-and-pages bot commented Mar 18, 2025

Deploying nmrium with  Cloudflare Pages  Cloudflare Pages

Latest commit: 07f0118
Status: ✅  Deploy successful!
Preview URL: https://6625fb6a.nmrium.pages.dev
Branch Preview URL: https://fix-adding-peak-crash-advanc.nmrium.pages.dev

View logs

@hamed-musallam hamed-musallam force-pushed the fix-adding-peak-crash-advance-mode branch from 910859a to 5d96da2 Compare March 19, 2025 10:22
@hamed-musallam hamed-musallam marked this pull request as ready for review March 19, 2025 10:22
@hamed-musallam hamed-musallam force-pushed the fix-adding-peak-crash-advance-mode branch from 5d96da2 to dda1f10 Compare March 19, 2025 10:25
@hamed-musallam hamed-musallam force-pushed the fix-adding-peak-crash-advance-mode branch from dda1f10 to 07f0118 Compare March 19, 2025 12:35
@targos
Copy link
Member

targos commented Mar 21, 2025

Is this ready for review?

@lpatiny
Copy link
Member

lpatiny commented Mar 24, 2025

@hamed-musallam Does this close an issue ?

@hamed-musallam
Copy link
Member Author

Is this ready for review?

I'm waiting for Luc to test it

@hamed-musallam
Copy link
Member Author

@hamed-musallam Does this close an issue ?

Yes, it is supposed to. did you test it again? i'm not Ok with appending the singlet and Massive to the list of couplings

@hamed-musallam hamed-musallam merged commit b7b9c68 into main Mar 24, 2025
10 checks passed
@hamed-musallam hamed-musallam deleted the fix-adding-peak-crash-advance-mode branch March 24, 2025 10:29
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.

Adding a peak crash
3 participants