[sync] feat(t5254): add record history filters for T5254#3370
[sync] feat(t5254): add record history filters for T5254#3370tea-artist wants to merge 1 commit into
Conversation
Synced from teableio/teable-ee@5a068c1 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>
|
|
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 3306cc1d8e
ℹ️ 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".
|
|
||
| try { | ||
| if (importRecords) { | ||
| const usersByEmail = await this.getSpaceUsersByEmail(ro.spaceId); |
There was a problem hiding this comment.
When baseId is provided, the import target is the existing base loaded above, but collaborator/user-field conversion still builds its email map from ro.spaceId supplied by the request. If that request value is stale or points at another space, Airtable collaborator cells are resolved against the wrong space, so imported user values can be dropped or mapped to users outside the target base's space. Use the resolved target base.spaceId here instead.
Useful? React with 👍 / 👎.
2 participants
🔄 Automated sync from EE repository.
136 commit(s) synced since last sync.
Authors
Included commits
ai-agent-engineT5222 T3933 (Pengap)Latest source commit: teableio/teable-ee@5a068c1
This PR was automatically created by the sync workflow.