Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Insert Default - Inserting ActiveModel with all attributes NotSet #432

Merged
merged 5 commits into from
Feb 6, 2022
Merged

Insert Default - Inserting ActiveModel with all attributes NotSet #432

merged 5 commits into from
Feb 6, 2022

Conversation

billy1624
Copy link
Member

@billy1624 billy1624 commented Jan 6, 2022 •
edited
Loading

@billy1624 billy1624 self-assigned this Jan 6, 2022
@billy1624 billy1624 mentioned this pull request Jan 6, 2022
Merged
@billy1624 billy1624 force-pushed the insert-default-values branch from 3d82e0d to e14e918 Compare January 6, 2022 14:23
@billy1624 billy1624 marked this pull request as ready for review January 6, 2022 15:34
@billy1624 billy1624 requested a review from tyt2y3 January 7, 2022 03:39
billy1624 added a commit to SeaQL/seaql.github.io that referenced this pull request Jan 31, 2022
@billy1624
Insert one with default (SeaQL/sea-orm#432)
88b6b3c
@tyt2y3 tyt2y3 merged commit ac0e145 into master Feb 6, 2022
@tyt2y3 tyt2y3 deleted the insert-default-values branch February 6, 2022 06:32
tyt2y3 added a commit that referenced this pull request Feb 6, 2022
@tyt2y3 @billy1624
Insert Default - Inserting ActiveModel with all attributes NotSet (…
2561bf4 
…#432)

* feat: apply alias on `ColumnRef::SchemaTableColumn`

* build: update sea-query dependency

* feat: insert default

* Use sea-query 0.21

Co-authored-by: Billy Chan <ccw.billy.123@gmail.com>
tyt2y3 added a commit that referenced this pull request Feb 6, 2022
@tyt2y3 @billy1624
Insert Default - Inserting ActiveModel with all attributes NotSet (…
7ce941b 
…#432)

* feat: apply alias on `ColumnRef::SchemaTableColumn`

* build: update sea-query dependency

* feat: insert default

* Use sea-query 0.21

Co-authored-by: Billy Chan <ccw.billy.123@gmail.com>
tyt2y3 added a commit that referenced this pull request Feb 6, 2022
@tyt2y3 @billy1624
Insert Default - Inserting ActiveModel with all attributes NotSet (…
33a87d7 
…#432)

* feat: apply alias on `ColumnRef::SchemaTableColumn`

* build: update sea-query dependency

* feat: insert default

* Use sea-query 0.21

Co-authored-by: Billy Chan <ccw.billy.123@gmail.com>
@tyt2y3
Copy link
Member

tyt2y3 commented Feb 6, 2022

Oh I forgot to merge the sea-query PR

tyt2y3 added a commit that referenced this pull request Feb 6, 2022
@tyt2y3
Revert "Insert Default - Inserting ActiveModel with all attributes …
b8a7ae7 
…`NotSet` (#432)"

This reverts commit 33a87d7.
@tyt2y3
Copy link
Member

tyt2y3 commented Feb 6, 2022

I don't think this feature is ready. I reverted the commit for now. Please revisit later.

billy1624 added a commit to SeaQL/seaql.github.io that referenced this pull request Feb 7, 2022
@billy1624
Revert "Insert one with default (SeaQL/sea-orm#432)"
26ac2d6 
This reverts commit 88b6b3c.
tyt2y3 added a commit that referenced this pull request Mar 6, 2022
@tyt2y3 @sunli829
@billy1624 @smrtrfszm @tasn @charleschege @cemoktra
Pr/471 (#583)
566c8c6 
* Add Poem example

* Name conflict of foreign key constraints when two entities have more than one foreign keys (#417)

* fix: name conflict of foreign key constraints when two entities have more than one fk

* test: update test case's foreign keys

* feat: override default name of foreign key constraint

* Add `max_lifetime` connection option

* Remove `r#` prefix from column name when field has a raw identifier

* Fix FromQueryResult when Result is redefined

Macros should only ever use absolute import paths.
Shorthands can clash (as they have in my case).

* Support the use of chrono::DateTime<Utc> in sea-orm

Add documentation for this

Temporarily use a fork to include new Sea-query code

Add tests for DateTimeUtc

Solve Github actions error by add the code to create a table

Assertion accuracy tests

Rectify incorrect format

* Refactoring

`ColumnDef` with default value

Cargo fmt

Update docs

Fixup

Support `DateTimeLocal`

Update docs

Codegen write db timestamp column as `DateTimeUtc` type

Update dependency

Merge branch 'sea-query/add-new-column-ref' into pr/429

feat: apply alias on `ColumnRef::SchemaTableColumn`

* Update sea-query dependency

* Fully quantity `std::result::Result` in proc_maros

* Migration (#335)

* Refactor `ConnectionTrait`

* Refactoring

* Build index & foreign key statements

* Fix imports

* Fixup

* Rocket example with migration

* async-std compatible with the tokio 1.0 runtime

* Use reexported dependency

* Compile without selecting any db backend

* Updating sea-orm-cli dep

* sea-orm-cli migrate commands

* cargo fmt

* Test [cli]

* Refactoring

* Clap app name should be "sea-orm-cli"

* Correctly capture MIGRATION_DIR

* Rename README

* Add `sea-orm-cli migrate init` command

* Update README

* Try restructured sea-query dependency (SeaQL/sea-schema#41)

* Set `DATABASE_URL` environment variable

* Fix sea-schema dependency

* Codegen `column_name` proc_macro attribute (#433)

* feat: codegen `column_name` proc_macro attribute

* test: codegen `column_name`

* Temporary fix

* Fix sea-orm dependency

* Insert Default - Inserting `ActiveModel` with all attributes `NotSet` (#432)

* feat: apply alias on `ColumnRef::SchemaTableColumn`

* build: update sea-query dependency

* feat: insert default

* Use sea-query 0.21

Co-authored-by: Billy Chan <ccw.billy.123@gmail.com>

* Easy joins with MockDatabase #447 (#455)

* Easy joins with MockDatabase #447

* fix MR

* add unit test

* Add test cases

* Cargo fmt

Co-authored-by: Bastian Schubert <bastian.schubert@crosscard.com>
Co-authored-by: Billy Chan <ccw.billy.123@gmail.com>
Co-authored-by: Chris Tsang <chris.2y3@outlook.com>

* cargo fmt

* Revert "Insert Default - Inserting `ActiveModel` with all attributes `NotSet` (#432)"

This reverts commit 33a87d7.

* Prepare release

* sea-orm-codegen 0.6.0

* sea-orm-cli 0.6.0

* sea-orm-macros 0.6.0

* 0.6.0

* Optimize GitHub actions

* Update examples sea-orm version

Update example sea-schema version

Update [cli] sea-schema version

* Fix [cli] cargo publish failed

* Update CHANGELOG

* Use sea-schema 0.5.1

* use sea_schema::migration::prelude

* Use caret requirements

* Fix build error

* Examples with migration (#509)

* Update examples sea-orm version

* Update example sea-schema version

* Update [cli] sea-schema version

* Fix [cli] cargo publish failed

* Update CHANGELOG

* Edit rocket example

* Poem example with migration

* Axum example with migration

* Refactoring

* Actix4 example with migration

* Actix example with migration

* Use sea_schema::migration::prelude

Co-authored-by: Sunli <scott_s829@163.com>
Co-authored-by: Billy Chan <30400950+billy1624@users.noreply.github.com>
Co-authored-by: Billy Chan <ccw.billy.123@gmail.com>
Co-authored-by: Szepesi Tibor <sztibi023@gmail.com>
Co-authored-by: Tom Hacohen <tasn@users.noreply.github.com>
Co-authored-by: Charles Chege <charleschege@protonmail.ch>
Co-authored-by: Bastian <b.schubert82@gmail.com>
Co-authored-by: Bastian Schubert <bastian.schubert@crosscard.com>
@billy1624 billy1624 mentioned this pull request Mar 8, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tyt2y3 tyt2y3 Awaiting requested review from tyt2y3

Assignees

@billy1624 billy1624

Labels
None yet
Projects
SeaQL Dev Tracker
Archived in project
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Failed to insert row with only default values
2 participants
@billy1624 @tyt2y3