Skip to content

feat: add IHasDspPresetSave interface for DSP preset management#1446

Merged
ndorin merged 3 commits into
mainfrom
feature-add-iHasDspPresetSave
Jul 14, 2026
Merged

feat: add IHasDspPresetSave interface for DSP preset management#1446
ndorin merged 3 commits into
mainfrom
feature-add-iHasDspPresetSave

Conversation

@jonnyarndt

Copy link
Copy Markdown
Contributor

Add PepperDash.Essentials.Core -> IHasDspPresetSave interface for DSP preset management

Copilot AI review requested due to automatic review settings July 13, 2026 23:03

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR introduces a new public interface in PepperDash.Essentials.Core intended to represent DSP devices that support saving presets in addition to recalling them, building on the existing IDspPresets contract.

Changes:

  • Added IHasDspPresetSave interface extending IDspPresets.
  • Introduced a save-by-key preset API on the new interface.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/PepperDash.Essentials.Core/Devices/IHasDspPresetSave.cs Outdated
Comment thread src/PepperDash.Essentials.Core/Devices/IHasDspPresetSave.cs Outdated
jonnyarndt and others added 2 commits July 13, 2026 16:08
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@jonnyarndt
jonnyarndt requested a review from ndorin July 13, 2026 23:22
@ndorin
ndorin merged commit 12828e8 into main Jul 14, 2026
2 checks passed
@ndorin
ndorin deleted the feature-add-iHasDspPresetSave branch July 14, 2026 17: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.

3 participants