Skip to content

Conversation

@murdybirdy
Copy link
Collaborator

No description provided.

};
const SpellList = withDatabase(withObservables([], ({ database }) => ({

const SpellList = (['id'], ({ database }) => ({
Copy link
Owner

@nathanielc nathanielc Nov 24, 2021

Choose a reason for hiding this comment

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

This part doesn't look quite right, the withDatabase and withOberservables function calls should still be in place.

The function calls should look like this:

const SpellList = withDatabase(withObservables([], ({ database }) => ({
    spells: database.collections.get('spells').query().observe(),
}))(_SpellList));

@nathanielc
Copy link
Owner

@murdybirdy I rebased the changes off my server changes and added the missing new bits that were needed because of my changes

This is good to merge

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.

3 participants