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

Cleaning up main.js / rendering / organization... #35

Closed
ivanstojic opened this issue Mar 23, 2015 · 3 comments
Closed

Cleaning up main.js / rendering / organization... #35

ivanstojic opened this issue Mar 23, 2015 · 3 comments

Comments

@ivanstojic
Copy link

Could we clean up the architecture a tad? I'm not the biggest fan of MV frameworks, but I figure a lil bit of templating and reengineering could be good for everyone who will want to contribute in the future?

The subreddit mentions that there's an Angular.js version somewhere awaiting a pull, and was suggesting that it'll be pulled soon. Any progress? Can I help somehow?

@SunburnedGoose
Copy link
Member

I've been working on the Angular port. I've broken up main.js and spread it
out across directives and services. The app is now responding to changes to
data binding.

What is not as much fun is writing unit tests. :/

When I get off my phone, I'll post the link to my branch.
On Sun, Mar 22, 2015 at 11:46 PM Ivan Stojic notifications@github.com
wrote:

Could we clean up the architecture a tad? I'm not the biggest fan of MV
frameworks, but I figure a lil bit of templating and reengineering could be
good for everyone who will want to contribute in the future?

The subreddit mentions that there's an Angular.js version somewhere
awaiting a pull, and was suggesting that it'll be pulled soon. Any
progress? Can I help somehow?


Reply to this email directly or view it on GitHub
#35.

@kyleshay
Copy link
Member

@SunburnedGoose
Copy link
Member

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

No branches or pull requests

3 participants