-
Notifications
You must be signed in to change notification settings - Fork 1.7k
Enh: Add Global ID #6718
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.
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
base: develop
Are you sure you want to change the base?
Enh: Add Global ID #6718
Conversation
1607cfe to
4b39bf5
Compare
4b39bf5 to
227583d
Compare
|
@luke- there is a theoretical question that comes along with the introduction of GID that is independent of the review of this PR or implementing code:
I would recommend the second approach. Would you see such a requirement (that plugins create updating-FKs before the update of the core version) as feasible? Maybe the core updater would have to check the maximum required versions of all enabled modules before an update is possible. The update process for any table that will become an
(I hope I was able express the challenge in an understandable way.) |
|
@martin-rueegg Thank you for your work here. I need some time to dig into the topic. |
ContentContainer,User,Space,ContentContainerActiveRecordWhat kind of change does this PR introduce?
Does this PR introduce a breaking change?
If yes, please describe the impact and migration path for existing applications:
The PR fulfills these requirements:
developbranch, not themasterbranch if no hotfixFix #xxx[,#xxx], where "xxx" is the Github issue number)If adding a new feature, the PR's description includes:
Other information:
Yii::$app->currentMigration(ignore for now) #6719(is|set)(Database)Installed()toApplicationTrait#6720