-
Notifications
You must be signed in to change notification settings - Fork 8
feat: Console SDK update for version 3.1.0 #65
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changes from 3 commits
c3c1c7d
fb3594a
8763798
3cea8e3
b86d0b2
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,21 @@ | ||
| export enum AppwriteMigrationResource { | ||
| User = 'user', | ||
| Team = 'team', | ||
| Membership = 'membership', | ||
| Database = 'database', | ||
| Table = 'table', | ||
| Column = 'column', | ||
| Index = 'index', | ||
| Row = 'row', | ||
| Document = 'document', | ||
| Attribute = 'attribute', | ||
| Collection = 'collection', | ||
| Bucket = 'bucket', | ||
| File = 'file', | ||
| Function = 'function', | ||
| Deployment = 'deployment', | ||
| Environmentvariable = 'environment-variable', | ||
| Site = 'site', | ||
| Sitedeployment = 'site-deployment', | ||
| Sitevariable = 'site-variable', | ||
| } | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,12 @@ | ||
| export enum FirebaseMigrationResource { | ||
| User = 'user', | ||
| Database = 'database', | ||
| Table = 'table', | ||
| Column = 'column', | ||
| Row = 'row', | ||
| Document = 'document', | ||
| Attribute = 'attribute', | ||
| Collection = 'collection', | ||
| Bucket = 'bucket', | ||
| File = 'file', | ||
| } | ||
|
Comment on lines
+1
to
+12
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 🧩 Analysis chain🏁 Script executed: fd 'migration-resource.ts' src/enums --exec cat {}Repository: appwrite/sdk-for-console Length of output: 1736 🌐 Web query:
💡 Result: Here are the key resource types / resource-name formats you’ll typically use when migrating or integrating Firebase Realtime Database ↔ Firestore via APIs: Firestore
Firebase Realtime Database
Migration guidance (RTDB → Firestore)
Sources: [1] [2] [3] [4] 🌐 Web query:
💡 Result: For Appwrite Firebase migrations ( In practice, the Firebase migration supports migrating these resource categories [2]:
Sources: the migrations endpoint definition/validation [1] and the migrations docs support matrix [2]. Remove or validate The Appwrite Firebase migration API only supports: 🤖 Prompt for AI Agents |
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,13 @@ | ||
| export enum SupabaseMigrationResource { | ||
| User = 'user', | ||
| Database = 'database', | ||
| Table = 'table', | ||
| Column = 'column', | ||
| Index = 'index', | ||
| Row = 'row', | ||
| Document = 'document', | ||
| Attribute = 'attribute', | ||
| Collection = 'collection', | ||
| Bucket = 'bucket', | ||
| File = 'file', | ||
| } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Enum member keys
Environmentvariable,Sitedeployment, andSitevariableshould be PascalCase.Every other member in this enum (and across
NHostMigrationResource/SupabaseMigrationResource) uses PascalCase. These three compound-word keys are inconsistently cased and will be visible as part of the public API. The string values (API literals) don't need to change — only the TypeScript identifiers.🔤 Proposed fix
📝 Committable suggestion
🤖 Prompt for AI Agents