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.

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

No need to place 'Embedded Objects' within Configuration <SchemaObject>[ #1001

Open
dotjon0 opened this issue Oct 31, 2022 · 7 comments · May be fixed by #1424
Open

No need to place 'Embedded Objects' within Configuration <SchemaObject>[ #1001

dotjon0 opened this issue Oct 31, 2022 · 7 comments · May be fixed by #1424

Comments

@dotjon0
Copy link

dotjon0 commented Oct 31, 2022

Description

'embedded objects' currently have to be included within the Configuration <SchemaObject>[. This feature request is to support not having to place 'embedded objects' within the Configuration <SchemaObject>[. This will increase maintainability and readability - a database with 70 models could easily have a few hundred 'embedded objects'...

Here is a related thread which provides an example #662 for context:

@RealmModel()
@mapto('person')
class _Person {
late String name;

@mapto('address')
_Address? personAddress;
}

// The generated Address class will be an embedded object.
@RealmModel(ObjectType.embeddedObject)
class _Address {
late String street;
late String city;
}


### How important is this improvement for you?
Would be a major improvement

@nielsenko
Copy link
Contributor

Actually, any dependency (embedded or not) could be automatically included, when adding a schema to a configuration, if we calculated the transitive dependencies in the generator.

@dotjon0
Copy link
Author

dotjon0 commented Oct 31, 2022

that would be amazing @nielsenko, sounds perfect!

@dotjon0
Copy link
Author

dotjon0 commented Nov 2, 2023

any updates on this?

@nirinchev
Copy link
Member

No, this is not on our immediate Todo list as it's a nice to have feature, but doesn't unlock any new use cases.

@dotjon0
Copy link
Author

dotjon0 commented Nov 2, 2023

Hi @nielsenko thanks for the update. For us, we have around 100 models, and around 300 embedded models.... So for us this is not a nice to have feature, its an essential to make this manageable...

@nielsenko
Copy link
Contributor

Hi @dotjon0. Credit for update goes to @nirinchev .. I guess over github usernames are a bit too similar 😄

@nirinchev
Copy link
Member

@dotjon0 fair enough - as is the case with most of these features - if it's something you deem critical for your project, definitely bring it up with your AE. They have mechanisms to influence the product roadmap and would be in best position to advise engineering on what features would have the broadest impact across our customers.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants