Collection update. #119
Collection update. #119
Conversation
lib/src/models/field.dart
Outdated
| @@ -70,6 +71,28 @@ class Field extends Equatable { | |||
| List<Object> get props => [name, type, isMultivalued]; | |||
| } | |||
|
|
|||
| class DroppableField extends Field { | |||
There was a problem hiding this comment.
Hey @kishorenc
Since only fields can be updated as of now, I've created DroppableField class to be used in the update function.
There was a problem hiding this comment.
@happy-san I think having an UpdateSchema class as detailed in the issue is better because we can add other updatable fields to it in future. By using a List< DroppableField> that flexibility is lost. With UpdateSchema, the update operation will look like this:
final updateSchema = UpdateSchema(
{
Field('company_category', Type.string),
Field('num_employees', drop: true)
}
);
await client.collection('companies').update(updateSchema);
This also means that we need to add a drop property to the field. We debated the semantics of doing this, but any alternative way of representing this became way too verbose.
There was a problem hiding this comment.
Makes sense, I'll update this accordingly.
| } | ||
| } | ||
|
|
||
| class CollectionUpdateSchema extends BaseSchema { |
There was a problem hiding this comment.
@kishorenc I've added CollectionUpdateSchema.
Change Summary
Support collection update operation.
#118
PR Checklist