Skip to content
This repository was archived by the owner on Oct 22, 2026. It is now read-only.

Keep the scripted refund and leave the restart queued - #75

Merged
SafetyMP merged 1 commit into
mainfrom
narrow/company-day
Oct 11, 2026
Merged

Keep the scripted refund and leave the restart queued#75
SafetyMP merged 1 commit into
mainfrom
narrow/company-day

Conversation

@SafetyMP

@SafetyMP SafetyMP commented Oct 11, 2026

Copy link
Copy Markdown
Owner

Summary

  • POST /api/company-day ignores autoApproveException, so an unauthenticated body cannot settle the restart.
  • The scripted day leaves the $49 refund on the ledger.
  • The default scenario is successful when that refund is held and the restart is pending.

Test plan

  • ./scripts/harness/verify.sh
  • npm run scenario

Made with Cursor

Copilot AI balanced review requested due to automatic review settings October 11, 2026 13:22
Copilot AI reviewed Oct 11, 2026

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@SafetyMP
SafetyMP merged commit bb01cbe into main Oct 11, 2026
3 checks passed
@SafetyMP
SafetyMP deleted the narrow/company-day branch October 11, 2026 13:23
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

Copilot code review Copilot
Copilot review effort
Applies to this pull request for everyone.Learn more about Copilot code review.
Copilot left review comments

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@SafetyMP