Conversation
Greptile SummaryThis PR migrates the Appwrite migration source from the legacy Confidence Score: 5/5Safe to merge — all changes are targeted SDK method renames with no logic regressions introduced. No P0/P1 issues found. Previously raised concerns (stale $collection unset, attributes fallback) have been explained or pre-exist the PR. The new code correctly handles the TablesDB API surface. No files require special attention. Important Files Changed
Reviews (5): Last reviewed commit: "updated" | Re-trigger Greptile |
| 'name' => $table['name'], | ||
| 'documentSecurity' => $table['documentSecurity'], | ||
| 'rowSecurity' => $rowSecurity, | ||
| 'documentSecurity' => $rowSecurity, |
There was a problem hiding this comment.
I think we only need rowSecurity
There was a problem hiding this comment.
we need the documentsecurity as well -> it was missing earlier. Now here internally it will get replaced due to the fromArray. So passing both rowSecurity and documentSecurity.
TablesDB is backward compatible but that is the not case with the legacy databases