Skip to content

fix: refresh $updatedAt after inline row edits - #3170

Merged
ChiragAgg5k merged 3 commits into
mainfrom
fix-2873-inline-updated-at
Aug 19, 2026
Merged

fix: refresh $updatedAt after inline row edits#3170
ChiragAgg5k merged 3 commits into
mainfrom
fix-2873-inline-updated-at

Conversation

@cursor

@cursor cursor Bot commented Aug 19, 2026

Copy link
Copy Markdown
Contributor

What does this PR do?

Inline spreadsheet edits called tablesDB.updateRow but kept the pre-write row in local grid state, so $updatedAt never changed until a full reload. Apply the API response system fields ($updatedAt, $permissions) before updating the grid.

Fixes #2873

Test Plan

  • Manually: edit a cell inline in TablesDB spreadsheet and confirm $updatedAt updates immediately
  • Sidebar "Update" path already invalidates and was unaffected

Related PRs and Issues

Open in WebView Automation

Inline spreadsheet edits called updateRow but kept the pre-write row in
local state, so $updatedAt never changed until a full reload. Apply the
API response system fields before updating the grid (console#2873).

Co-authored-by: chiragaggarwal5k <chiragaggarwal5k@gmail.com>
@appwrite

appwrite Bot commented Aug 19, 2026

Copy link
Copy Markdown

Console (appwrite/console)

Project ID: 688b7bf400350cbd60e9

Sites (1)
Site Status Logs Preview QR
console-stage
688b7cf6003b1842c9dc
Ready Ready View Logs Preview URL QR Code

Tip

Build commands execute in runtime containers during deployment

@cursor
cursor Bot requested a review from ChiragAgg5k August 19, 2026 03:35
@cursor
cursor Bot marked this pull request as ready for review August 19, 2026 03:35
@greptile-apps

greptile-apps Bot commented Aug 19, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

The PR refreshes locally cached system fields after a successful inline TablesDB row edit.

  • Uses the updateRow response to refresh $updatedAt and $permissions.
  • Updates the paginated spreadsheet state with the refreshed row.
  • Preserves the callback鈥檚 existing boolean success contract.

Confidence Score: 5/5

The PR appears safe to merge.

No blocking failure remains.

Important Files Changed

Filename Overview
src/routes/(console)/project-[region]-[project]/databases/database-[database]/table-[table]/spreadsheet.svelte Applies server-returned system fields to the edited row and propagates the successful result into paginated grid state.

Reviews (3): Last reviewed commit: "chore: nudge CI for stuck Tests queue" | Re-trigger Greptile

cursoragent and others added 2 commits August 19, 2026 03:44
Co-authored-by: chiragaggarwal5k <chiragaggarwal5k@gmail.com>
Co-authored-by: chiragaggarwal5k <chiragaggarwal5k@gmail.com>

@ChiragAgg5k ChiragAgg5k left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Confirmed this is a console-only bug: updateRow already returns a new $updatedAt, but inline edit ignores the response and keeps the pre-write row in the grid. Applying the API system fields before paginatedRows.update is the right fix.

@ChiragAgg5k
ChiragAgg5k merged commit 14a0c36 into main Aug 19, 2026
4 checks passed
@ChiragAgg5k
ChiragAgg5k deleted the fix-2873-inline-updated-at branch August 19, 2026 05:31
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.

Console - Databases - inline updating a row column does not update $updatedAt

2 participants