[sync] T5439 Fix computed seed-all table loading#3441
Conversation
Synced from teableio/teable-ee@903a42a Co-authored-by: Aries X <caoxing9@gmail.com> Co-authored-by: Bieber <artist@teable.io> Co-authored-by: Boris <boris2code@outlook.com> Co-authored-by: Jocky Zhou <jocky@teable.ai> Co-authored-by: Jun Lu <hammond@teable.io> Co-authored-by: Pengap <penganpingprivte@gmail.com> Co-authored-by: SkyHuang <sky.huang.fe@gmail.com> Co-authored-by: Uno <uno@teable.ai> Co-authored-by: nichenqin <nichenqin@hotmail.com>
|
|
1 similar comment
|
|
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 04cd9db4d3
ℹ️ 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".
| tableId, | ||
| fieldIds | ||
| ); | ||
| await this.recordOpenApiV2Service.updateRecords(tableId, updatePayload); |
There was a problem hiding this comment.
Skip empty update batches before creating pasted records
When paste-by-id targets no existing records, such as pasting into an empty table or below the current rows, buildPasteByIdPayload returns updatePayload.records as an empty array and puts the pasted rows in createPayload. This V2 path calls updateRecords first, but UpdateRecordsCommand rejects explicit records: [] with its min(1) validation, so the request fails before the new records are created. Please skip the update call when there are no records to update.
Useful? React with 👍 / 👎.
🧹 Preview Environment Cleanup
|
🔄 Automated sync from EE repository.
206 commit(s) synced since last sync.
Authors
Included commits
ai-agent-engineT5222 T3933 (Pengap)Latest source commit: teableio/teable-ee@903a42a
This PR was automatically created by the sync workflow.