Fix: overriding timestamps on row duplicate action

[ItzNotABug]

What does this PR do?

(Provide a description of what this PR does.)

Test Plan

(Write your test plan here. If you changed any code, please provide us with clear instructions on how you verified your changes work.)

Related PRs and Issues

(If this PR is related to any other PR or resolves any issue or related to any issue link all related PR and issues here.)

Have you read the Contributing Guidelines on issues?

(Write your answer here.)

Summary by CodeRabbit

• New Features
  • Duplicating a table row now pre-fills the create-row sheet with partial data while automatically excluding system-managed date fields (e.g., created/updated timestamps). This streamlines duplication, avoids unintended carryover of timestamps, and ensures cleaner starting values. The sheet accepts partial entries for easier adjustment before saving. No other user-facing behavior changes.

[coderabbitai]

Walkthrough

The Svelte component at src/routes/(console)/project-[region]-[project]/databases/database-[database]/table-[table]/spreadsheet.svelte updates the exported prop showRowCreateSheet, changing its row type from Models.Row | null to Partial<Models.Row> | null. The duplication flow now constructs a rowWithoutDates by removing createdAt and updatedAt from the source row and assigns this partial object to showRowCreateSheet.row for pre-filling the create-row sheet. No other behavior changes are indicated.

Suggested reviewers

• TorstenDittmann

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check	✅ Passed	The title clearly identifies the core fix—addressing timestamp handling on the row duplication flow— which directly corresponds to the change of stripping createdAt/updatedAt fields and updating the prop type to Partial<Models.Row> to prevent unintended timestamp overrides. It is concise, specific to the duplication action, and conveys the primary developer intent without superfluous details.
Docstring Coverage	✅ Passed	No functions found in the changes. Docstring coverage check skipped.

✨ Finishing touches

🧪 Generate unit tests

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch fix-dat-762

---

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between e8b9341 and 21ed4cd.

📒 Files selected for processing (1)

• src/routes/(console)/project-[region]-[project]/databases/database-[database]/table-[table]/spreadsheet.svelte (2 hunks)

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (2)

• GitHub Check: e2e
• GitHub Check: build

🔇 Additional comments (2)

src/routes/(console)/project-[region]-[project]/databases/database-[database]/table-[table]/spreadsheet.svelte (2)

545-554: LGTM! Correctly strips timestamps on row duplication.

The destructuring approach cleanly removes $createdAt and $updatedAt before pre-filling the create sheet, preventing timestamp override issues. The implementation is idiomatic and aligns with the PR objective.

---

98-101: Ensure existingData prop handles partial rows

The bind:existingData={$showRowCreateSheet.row} in +page.svelte (line 268) now passes a Partial<Models.Row>. Confirm that the existingData prop on the target component is updated to accept Partial<Models.Row> | null.

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}