Add _ID automatic to models

[jaberbu]

Would be great not make mandatory to add _id field to Interfaces

interface Schema { _id?: string | undefined; FIELD_1: string; FIELD_2: number; }

[jaberbu]

Options: Results:

But if you want access to first element you need to know its id: results['1ARo7ZoCFcwETOtVjjAI'].BRAND

[ezequieltejada]

I'll take it

[jaberbu]

@ezequieltejada

The idea is add _id automatically to the interface created by the user.

Interface created buy user and passed to Onfirework<Schema>('COLLECTION')

And gets _id too

[ezequieltejada]

Ups! Sorry, I miss understood. Please reject the pull request

[jaberbu]

Close #20

[jaberbu]

@ezequieltejada bad news :( works 50%

Works great as Filter suggest 👍

Not work 👎

We need reopen an issue to try solved this bug.

[jaberbu]

There is also the problem that requires importing Results to declared filters outside the listDocs(where)

const where:Filter<Result<BikeSchema>>[] = [
    ['BRAND', '==', 'Ducati']
]

[ezequieltejada]

I'll take a look on it

[jaberbu]

@ezequieltejada We should revert type for param Filter to Filter<T>[] is not necesary filtering by _ID

eg: bikes.listDocs([['_id', '==', '1234']])

This is other thing:

const where:Filter<Result<BikeSchema>>[] = [['BRAND', '==', 'Ducati']]
const ducati:Promise<Result<BikeSchema>[]> = bike.listDocs(where)

[jaberbu]

Fixed in Pull Request #22