Skip to content

Validate dashboard checkpoint ids#66

Open
w287346141 wants to merge 1 commit into
Yapie0:mainfrom
w287346141:codex/validate-dashboard-checkpoint-id
Open

Validate dashboard checkpoint ids#66
w287346141 wants to merge 1 commit into
Yapie0:mainfrom
w287346141:codex/validate-dashboard-checkpoint-id

Conversation

@w287346141
Copy link
Copy Markdown

Summary

  • reject non-string checkpoint restore ids with a 400 response
  • reject non-string checkpoint delete ids with a 400 response
  • add focused dashboard checkpoint id validation tests

Validation

  • npm.cmd test -- --run tests/dashboard-checkpoint-id.test.ts tests/server-dashboard.test.ts
  • npm.cmd run lint
  • npm.cmd run typecheck
  • git diff --check
  • push hook ran npm.cmd run verify

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.

1 participant