Skip to content

Conversation

@nalenz-objectbox
Copy link
Contributor

See #18.

Copy link
Contributor

@vaind vaind left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

While dart can be run as a script, it's supposed to be compiled for "dartaotruntime" for production. Thus it can't really just read the model.json from the current directory. Or am I missing something?

All the other bindings handle this by including all necessary information (ID, UID, Type in DB) in the generated files.

However, it seems the model file could be embedded in a package with - https://pub.dev/packages/resource.

Since that still comes with a performance & size drawback, consider whether using parts of the original generator code would make sense, thus completely avoiding having to deal with resources and possibly two ways of loading the file (compiled vs in VM)

@vaind
Copy link
Contributor

vaind commented Sep 26, 2019

Also don't forget to rebase/merge changes from dev

@vaind
Copy link
Contributor

vaind commented Oct 2, 2019

The remaining things before this PR can be merged:

@Buggaboo
Copy link
Contributor

Buggaboo commented Oct 2, 2019

I'm currently working on doing Queries, is the boolean / byte / double / long implementation for generating properties included in this branch?

@vaind vaind mentioned this pull request Oct 3, 2019
3 tasks
# Conflicts:
#	test/basics_test.dart
#	test/box_test.dart
#	test/test.dart
@vaind vaind force-pushed the 18-automatic-id-generation branch from c46a64a to ab451b6 Compare October 3, 2019 13:27
@vaind vaind force-pushed the 18-automatic-id-generation branch from ab451b6 to 956f5a8 Compare October 3, 2019 13:33
@vaind
Copy link
Contributor

vaind commented Oct 3, 2019

Merging this into dev, please add the missing class docs directly on dev branch @nalenz-objectbox

@vaind vaind merged commit 7c5536d into dev Oct 3, 2019
@vaind vaind deleted the 18-automatic-id-generation branch October 3, 2019 14:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants