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

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

Merged
merged 3 commits into from
Jan 27, 2022
Merged

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

merged 3 commits into from
Jan 27, 2022

Conversation

billy1624
Copy link
Member

@billy1624 billy1624 self-assigned this Jan 3, 2022
@billy1624 billy1624 marked this pull request as ready for review January 3, 2022 08:52
@billy1624 billy1624 mentioned this pull request Jan 18, 2022
Closed
@tyt2y3 tyt2y3 merged commit 76c0d7f into master Jan 27, 2022
@tyt2y3 tyt2y3 deleted the fk-name-conflicts branch January 27, 2022 17:03
@tyt2y3
Copy link
Member

tyt2y3 commented Jan 27, 2022

Nice

billy1624 added a commit to SeaQL/seaql.github.io that referenced this pull request Jan 31, 2022
@billy1624
Update create_table_from_entity's foreign key name (SeaQL/sea-orm#417)
1cf4cd3
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 <[email protected]>

* 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 <[email protected]>
Co-authored-by: Billy Chan <[email protected]>
Co-authored-by: Chris Tsang <[email protected]>

* 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 <[email protected]>
Co-authored-by: Billy Chan <[email protected]>
Co-authored-by: Billy Chan <[email protected]>
Co-authored-by: Szepesi Tibor <[email protected]>
Co-authored-by: Tom Hacohen <[email protected]>
Co-authored-by: Charles Chege <[email protected]>
Co-authored-by: Bastian <[email protected]>
Co-authored-by: Bastian Schubert <[email protected]>
This was referenced Apr 25, 2022
Closed
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
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.

Foreign key constraint collision when multiple keys exist between the same two tables
2 participants
@billy1624 @tyt2y3