Skip to content
This repository has been archived by the owner on Jul 16, 2023. It is now read-only.

Fixing bug introduced by auto-discovery #314

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

igorsantos07
Copy link
Member

@igorsantos07 igorsantos07 commented Oct 9, 2018

@lloricode, could you please use version dev-fix-autodiscovery and check if it all works fine now?
Make sure you try to instantiate an Ardent model and run a query on it. Unfortunately, I don't have a working project to test this right now.

@igorsantos07 igorsantos07 changed the title Fixing bug introduced by auto-discovery from #305. Fixes #312 Fixing bug introduced by auto-discovery Oct 9, 2018
@lloricode
Copy link

@igorsantos07 i will :)

@lloricode
Copy link

My unit testing seems good, with one model extends on it. I will update you later after all my models extend to ardent.

Thanks :)

@spyp
Copy link
Collaborator

spyp commented Oct 9, 2018

In 5.7 when i installed i get problem that $this-package is not defined in Serviceprovider
i wonder.i'm testing now. I think we need new branch for 5.7 and above to re build this package.

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

Successfully merging this pull request may close these issues.

3 participants