Skip to content

Conversation

harikt
Copy link
Member

@harikt harikt commented Feb 14, 2016

This allows to pass the map array and paths to the template registry on constructor using the ViewFactory class.

@pmjones
Copy link
Member

pmjones commented Oct 3, 2016

Hey @harikt -- can you add some README documentation on this? Then I can merge it and make a minor release.

@pmjones
Copy link
Member

pmjones commented Oct 3, 2016

Or you can merge it and make the release. :-)

@harikt
Copy link
Member Author

harikt commented Oct 4, 2016

Sure, I will update the README.md . I don't like me sending a PR and merging the same for a few reasons. If the same person is reviewing the possibility of missing things are greater. So I prefer anyone other than me looking into it and verifying the same. So it is always for your consideration 👍 . If you put a +1 I will do the next steps ;) .

@harikt
Copy link
Member Author

harikt commented Oct 4, 2016

@pmjones have added some information to README. Let me know if that helps.

Thank you

<?php
$view_factory = new \Aura\View\ViewFactory;
$view = $view_factory->newInstance(
$helpers = null,
Copy link
Member

Choose a reason for hiding this comment

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

Shouldn't this just be null, rather than an assignment?

Copy link
Member Author

Choose a reason for hiding this comment

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

You are right, it can be just null. But I wanted to show it is helpers and people can pass helpers other than null.

@harikt
Copy link
Member Author

harikt commented Oct 13, 2016

@pmjones in case you have some time. Have a look into the updated docs.

I will make another release for Aura.View once you have reviewed the same.

Thank you.

@harikt
Copy link
Member Author

harikt commented Feb 2, 2017

@pmjones ping.

@pmjones pmjones merged commit ed8960f into auraphp:2.x Feb 2, 2017
@pmjones
Copy link
Member

pmjones commented Feb 2, 2017

Done :-)

@harikt harikt deleted the feature branch February 2, 2017 18:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants