Skip to content

Document the cross-card-table card move recipe#419

Merged
jeremy merged 1 commit into
masterfrom
clarify-cross-card-table-card-move
Jul 23, 2026
Merged

Document the cross-card-table card move recipe#419
jeremy merged 1 commit into
masterfrom
clarify-cross-card-table-card-move

Conversation

@jeremy

@jeremy jeremy commented Jul 23, 2026

Copy link
Copy Markdown
Member

The Move a card section already mentioned that column_id can be a wormhole id, but left the reader to assemble the workflow. This spells it out in one place.

What's now documented:

  • Links the card table wormholes endpoints directly and gives the two-step recipe: create a wormhole on the source card table pointing at the destination column, then move the card onto the wormhole's id.
  • The destination is another card table — which may be in another project, but need not be.
  • Only wormholes with "linked": true accept a card; moving onto an unlinked one returns 404 Not Found.
  • A valid position does not control where a teleported card lands (placement is decided when the asynchronous move completes), but an invalid position still returns 400 Bad Request.
  • The 204 means the teleport was queued; completion is asynchronous, the response does not expose the destination card's new recording id, and the original card id returns 404 Not Found once the move completes.

Docs only — one file, no new endpoints.

Copilot AI review requested due to automatic review settings July 23, 2026 04:59

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.

Pull request overview

This PR expands the Move a card documentation to explicitly describe the two-step workflow for moving a card across card tables by using a wormhole id as column_id, clarifying key behaviors around linking, validation, and asynchronous completion.

Changes:

  • Adds a step-by-step “recipe” for cross-card-table moves via wormholes (create wormhole → move card onto wormhole).
  • Clarifies constraints/behaviors for wormhole moves (requires linked wormhole, async completion, position validation/limitations).
  • Improves discoverability by linking directly to the card table wormholes documentation.

Tip

If you aren't ready for review, convert to a draft PR.
Click "Convert to draft" or run gh pr ready --undo.
Click "Ready for review" or run gh pr ready to reengage.


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

Comment thread sections/card_table_cards.md Outdated
Comment thread sections/card_table_cards.md Outdated

@chatgpt-codex-connector chatgpt-codex-connector Bot 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 9c0280c5e2

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread sections/card_table_cards.md Outdated
Copilot AI review requested due to automatic review settings July 23, 2026 05:04
@jeremy
jeremy force-pushed the clarify-cross-card-table-card-move branch from 9c0280c to 46dbb3c Compare July 23, 2026 05:04

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.

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

Copilot AI review requested due to automatic review settings July 23, 2026 05:12
@jeremy
jeremy force-pushed the clarify-cross-card-table-card-move branch from 46dbb3c to 3f1f598 Compare July 23, 2026 05:12

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.

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

The Move a card section noted that column_id could be a wormhole id but
left the workflow for the reader to assemble. Spell it out: link the
card table wormholes endpoints directly and give the two-step recipe for
moving a card onto another card table, which may be in another project.

Clarify the contract that's easy to get wrong:

- Only linked wormholes accept a card; the card table's wormholes array
  also lists unlinked ones, and moving onto those returns 404 Not Found.
- A valid position does not control where a teleported card lands —
  placement is decided when the asynchronous move completes — but an
  invalid position still returns 400 Bad Request.
- The 204 means the teleport was queued; completion is asynchronous, the
  response does not expose the destination card's new id, and the
  original card id returns 404 once the move completes.
Copilot AI review requested due to automatic review settings July 23, 2026 05:17
@jeremy
jeremy force-pushed the clarify-cross-card-table-card-move branch from 3f1f598 to 1432ab8 Compare July 23, 2026 05:17

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.

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

@jeremy
jeremy merged commit 742e0ee into master Jul 23, 2026
1 check passed
@jeremy
jeremy deleted the clarify-cross-card-table-card-move branch July 23, 2026 05:25
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