Skip to content
This repository has been archived by the owner on Sep 12, 2019. It is now read-only.

Front Router 2.0 #6

Open
wants to merge 8 commits into
base: master
Choose a base branch
from
Open

Front Router 2.0 #6

wants to merge 8 commits into from

Conversation

gakimball
Copy link
Contributor

Goals for 2.0:

  • Create a FrontRouter class to bundle functionality
  • Usable without a Gulp stream
  • Command-line interface
  • Angular 2 output

@soumak77
Copy link

@gakimball would you have any interest in picking this up and committing it to https://github.com/base-apps/angular-front-router?

@gakimball
Copy link
Contributor Author

@soumak77 Most definitely, I'd love to help out. :)

@gakimball
Copy link
Contributor Author

@soumak77 Rebasing your changes into this branch is a little tricky, so I'm just going to add what I have here by hand.

@gakimball
Copy link
Contributor Author

Actually, which of these features specifically interest you for your fork? Most of these changes were designed to make Front Router multi-framework, but if you're mostly interested in keeping it to Angular 1.x then some of these changes aren't as necessary.

@soumak77
Copy link

@gakimball I like the idea to enable the ability to make it multi-framework, though the repository name would need to change. For now it just needs to support angular 1.x and a non-gulp stream, but in the future I would like to use it for angular 2 as well.

@soumak77
Copy link

@gakimball let me know what I can do to help with the merge

@gakimball
Copy link
Contributor Author

Does supporting older Node versions concern you? These libraries mostly support 0.10/0.12+, but supporting just 4.x+ would make some features of this library easier to write.

@soumak77
Copy link

Just make it support 4.x+. The new angular-base-apps template already has this requirement if I'm not mistaken.

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

Successfully merging this pull request may close these issues.

2 participants