[sync] ci: stop forcing v2 in community integration tests#3567
[sync] ci: stop forcing v2 in community integration tests#3567tea-artist wants to merge 1 commit into
Conversation
Synced from teableio/teable-ee@2cd6560 Co-authored-by: Aries X <caoxing9@gmail.com> Co-authored-by: Bieber <artist@teable.io> 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: 7787703ce0
ℹ️ 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".
| if ( | ||
| this.fieldMap[fieldId]?.type === FieldType.Link && | ||
| !this.isResolvedLinkCellValue(record.fields[fieldId]) | ||
| ) { | ||
| return false; |
There was a problem hiding this comment.
Let cleared link values sync back
When the update response contains a cleared link value (null or []) for any link field other than the one optimistically edited, this check treats it as “unresolved” and skips updateComputedField, so reciprocal/self-link cleanup returned by the server leaves the old linked titles in the client until a realtime or full refresh arrives. Only link payloads that still contain records without titles should be skipped; empty values need to be applied.
Useful? React with 👍 / 👎.
🧹 Preview Environment Cleanup
|
🔄 Automated sync from EE repository.
63 commit(s) synced since last sync.
Authors
Included commits
Latest source commit: teableio/teable-ee@2cd6560
This PR was automatically created by the sync workflow.