add typegoose models #1

Merged
eirk merged 6 commits from master into master 2022-02-28 11:51:51 -05:00
eirk commented 2021-12-25 14:02:34 -05:00 (Migrated from gitlab.libraryofcode.org)

add typegoose models

add typegoose models
eirk commented 2021-12-25 14:06:13 -05:00 (Migrated from gitlab.libraryofcode.org)

added 2 commits

Compare with previous version

added 2 commits <ul><li>b138bcb3 - fix typo</li><li>ca098cb9 - fix space</li></ul> [Compare with previous version](/engineering/community-relations/database/-/merge_requests/1/diffs?diff_id=553&start_sha=7f1640d9d4fb309a4bdbba05283478e8b166a301)
eirk commented 2021-12-25 14:09:32 -05:00 (Migrated from gitlab.libraryofcode.org)

added 1 commit

Compare with previous version

added 1 commit <ul><li>cad13f63 - better enum values</li></ul> [Compare with previous version](/engineering/community-relations/database/-/merge_requests/1/diffs?diff_id=555&start_sha=ca098cb9de56e299d7151715e3ebc28ad4c9c261)
matthew commented 2021-12-25 16:16:13 -05:00 (Migrated from gitlab.libraryofcode.org)

assigned to @hiroyuki

assigned to @hiroyuki
matthew commented 2021-12-25 16:30:13 -05:00 (Migrated from gitlab.libraryofcode.org)

Not required

Not required
matthew commented 2021-12-25 16:30:14 -05:00 (Migrated from gitlab.libraryofcode.org)

Can someone check if this field is required.

Can someone check if this field is required.
matthew commented 2021-12-25 16:30:15 -05:00 (Migrated from gitlab.libraryofcode.org)

This isn’t required but can we set the initial value to []

This isn’t required but can we set the initial value to []
matthew commented 2021-12-25 16:30:15 -05:00 (Migrated from gitlab.libraryofcode.org)

Not required

Not required
matthew commented 2021-12-25 16:30:16 -05:00 (Migrated from gitlab.libraryofcode.org)

Not required

Not required
matthew commented 2021-12-25 16:30:16 -05:00 (Migrated from gitlab.libraryofcode.org)

Not required.

Not required.
matthew commented 2021-12-25 16:30:16 -05:00 (Migrated from gitlab.libraryofcode.org)

I don’t believe this is required; logic check please

I don’t believe this is required; logic check please
hiroyuki commented 2021-12-27 19:43:56 -05:00 (Migrated from gitlab.libraryofcode.org)

Regarding the not required comments, I believe that eirk just marked as required where there was an absence of question marks (indicating an optional property).

Regarding the not required comments, I believe that eirk just marked as required where there was an absence of question marks (indicating an optional property).
matthew commented 2022-02-28 11:51:29 -05:00 (Migrated from gitlab.libraryofcode.org)

resolved all threads

resolved all threads
matthew commented 2022-02-28 11:51:51 -05:00 (Migrated from gitlab.libraryofcode.org)

mentioned in commit b5ca27650e

mentioned in commit b5ca27650e810a2e2d4a537c53e2bc971a8c8d57
Sign in to join this conversation.
No reviewers
No Milestone
No project
No Assignees
1 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: engineering/cr-database#1
There is no content yet.