Skip to content

Conversation

@samholmes
Copy link
Contributor

@samholmes samholmes commented Nov 7, 2025

CHANGELOG

Does this branch warrant an entry to the CHANGELOG?

  • Yes
  • No

Dependencies

none

Requirements

If you have made any visual changes to the GUI. Make sure you have:

  • Tested on iOS device
  • Tested on Android device
  • Tested on small-screen device (iPod Touch)
  • Tested on large-screen device (tablet)

Note

Adds remote ramp quote filtering using info-server rollup (via @edge.app/drupe) and updates dependencies/config.

  • Ramps:
    • Add remote filter in validateRampConstraintParams (src/plugins/ramps/rampConstraints.ts) using infoServerData.rollup.rampQuoteFilter evaluated via drupe(...); short‑circuit rejects before local constraints.
  • Dependencies:
    • Add @edge.app/drupe and bump edge-info-server to ^3.10.0.
  • Config/Docs:
    • Add .npmrc for JSR registry.
    • Update CHANGELOG.md noting remote ramp plugin filtering.

Written by Cursor Bugbot for commit b839865. This will update automatically on new commits. Configure here.


Copy link
Collaborator

@Jon-edge Jon-edge left a comment

Choose a reason for hiding this comment

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

Nice tidy impl, just fix the bugbot

@samholmes samholmes force-pushed the sam/remote-ramp-constraints branch 2 times, most recently from 00d9f40 to fa5b78f Compare November 10, 2025 23:09
@samholmes samholmes force-pushed the sam/remote-ramp-constraints branch from fa5b78f to 4880acc Compare November 12, 2025 23:22
Use `@edge.app/drupe` for mango-based constraints from the info-server.
@samholmes samholmes force-pushed the sam/remote-ramp-constraints branch from 4880acc to b839865 Compare November 13, 2025 20:47
@samholmes samholmes enabled auto-merge November 13, 2025 20:48
@samholmes samholmes merged commit 251912f into develop Nov 13, 2025
3 checks passed
@samholmes samholmes deleted the sam/remote-ramp-constraints branch November 13, 2025 21:31
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