Skip to content

refactor(spectra): submit payload for multi-save with spectra_list#2993

Open
Nicolass67 wants to merge 7 commits intomainfrom
refactor/spectra-save-payload-flow
Open

refactor(spectra): submit payload for multi-save with spectra_list#2993
Nicolass67 wants to merge 7 commits intomainfrom
refactor/spectra-save-payload-flow

Conversation

@Nicolass67
Copy link
Contributor

Summary

This PR updates submit payload handling to support multi-save in a consistent way.

  • Refactored the spectra save flow to make saveOp smaller and easier to read.
  • Split save logic into focused helpers (getSavePayloads, buildSerializedPayload, saveTarget, combineBatchIfNeeded).
  • Moved combine API call from ViewSpectra to SpectraActions.CombineSpectra for cleaner separation.
  • Kept combine generation only for accepted multi-layouts and only when batch conditions are met.
  • Use spectra_list as the submit payload format.
  • Apply the same submit flow for single and multi spectra.
  • Ensure each spectrum item contains its own required save data.

@PiTrem PiTrem changed the title Refactor submit payload for multi-save with spectra_list refactor: submit payload for multi-save with spectra_list Mar 4, 2026
@PiTrem PiTrem changed the title refactor: submit payload for multi-save with spectra_list refactor(spectra): submit payload for multi-save with spectra_list Mar 4, 2026
@Nicolass67 Nicolass67 force-pushed the refactor/spectra-save-payload-flow branch from 116b3de to fda81ea Compare March 4, 2026 11:59
Nicolass67 and others added 5 commits March 4, 2026 13:19
rebase

Update package.json

Update package.json

Update package.json

Update package.json

Update package.json

update package.json

Update package.json

Update package.json

Update package.json

Update package.json

Update package.json

Add attachment handling to ViewSpectra and update SaveToFile action

Update package.json

Revert package.json changes and remove public/pdf.worker.min.js

Made-with: Cursor

Remove resolutions section from package.json
@PiTrem PiTrem force-pushed the refactor/spectra-save-payload-flow branch from fda81ea to c026403 Compare March 4, 2026 12:21
@PiTrem PiTrem added this to the v3.1 milestone Mar 4, 2026
…re processing peaks, improving data integrity for curve indexing.
…ata handling for peaks, shifts, and integrations, improving compatibility with downstream functions.
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.

2 participants