{"action":"created","issue":{"url":"https://api.github.com/repos/meteor-useraccounts/core/issues/308","labels_url":"https://api.github.com/repos/meteor-useraccounts/core/issues/308/labels{/name}","comments_url":"https://api.github.com/repos/meteor-useraccounts/core/issues/308/comments","events_url":"https://api.github.com/repos/meteor-useraccounts/core/issues/308/events","html_url":"https://github.com/meteor-useraccounts/core/issues/308","id":60493262,"number":308,"title":"Don't hardly depend on iron:router - Flow Router Integration","user":{"login":"Sanjo","id":727955,"avatar_url":"https://avatars.githubusercontent.com/u/727955?v=3","gravatar_id":"","url":"https://api.github.com/users/Sanjo","html_url":"https://github.com/Sanjo","followers_url":"https://api.github.com/users/Sanjo/followers","following_url":"https://api.github.com/users/Sanjo/following{/other_user}","gists_url":"https://api.github.com/users/Sanjo/gists{/gist_id}","starred_url":"https://api.github.com/users/Sanjo/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Sanjo/subscriptions","organizations_url":"https://api.github.com/users/Sanjo/orgs","repos_url":"https://api.github.com/users/Sanjo/repos","events_url":"https://api.github.com/users/Sanjo/events{/privacy}","received_events_url":"https://api.github.com/users/Sanjo/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/meteor-useraccounts/core/labels/enhancement","name":"enhancement","color":"84b6eb"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":107,"created_at":"2015-03-10T12:40:53Z","updated_at":"2015-08-06T05:32:13Z","closed_at":null,"body":"I'd like to use the useraccounts package with the [flow-router](https://atmospherejs.com/meteorhacks/flow-router). Right now it's not possible to get rid of iron:router when using the useraccounts package because of the hard dependency on it.\r\n\r\n__My suggested solutions to fix this are either:__\r\n\r\n1. __Make iron:router specific features optional.__ They are only available when the app uses iron:router. This  can be achieved by making the dependency to iron:router weak and check the existence of `Package['iron:router']` for the iron:router specific parts.\r\n\r\n2. __Use polymorphism to support multiple routers.__ Create a common router interface object that will delegate the execution to the router that is included in the app. This would be iron:router, meteorhacks:flow-router or a \"no router\" implementation."},"comment":{"url":"https://api.github.com/repos/meteor-useraccounts/core/issues/comments/128249525","html_url":"https://github.com/meteor-useraccounts/core/issues/308#issuecomment-128249525","issue_url":"https://api.github.com/repos/meteor-useraccounts/core/issues/308","id":128249525,"user":{"login":"tjmonsi","id":2645718,"avatar_url":"https://avatars.githubusercontent.com/u/2645718?v=3","gravatar_id":"","url":"https://api.github.com/users/tjmonsi","html_url":"https://github.com/tjmonsi","followers_url":"https://api.github.com/users/tjmonsi/followers","following_url":"https://api.github.com/users/tjmonsi/following{/other_user}","gists_url":"https://api.github.com/users/tjmonsi/gists{/gist_id}","starred_url":"https://api.github.com/users/tjmonsi/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tjmonsi/subscriptions","organizations_url":"https://api.github.com/users/tjmonsi/orgs","repos_url":"https://api.github.com/users/tjmonsi/repos","events_url":"https://api.github.com/users/tjmonsi/events{/privacy}","received_events_url":"https://api.github.com/users/tjmonsi/received_events","type":"User","site_admin":false},"created_at":"2015-08-06T05:32:13Z","updated_at":"2015-08-06T05:32:13Z","body":"Here's what it is loading in my dependencies:\r\n\r\n```\r\niron:controller            added, version 1.0.8\r\niron:core                  added, version 1.0.8\r\niron:dynamic-template      added, version 1.0.8\r\niron:layout                added, version 1.0.8\r\niron:location              added, version 1.0.9\r\niron:middleware-stack      added, version 1.0.9\r\niron:router                added, version 1.0.9\r\niron:url                   added, version 1.0.9\r\nuseraccounts:core          added, version 1.11.1\r\nuseraccounts:flow-routing  added, version 1.11.1\r\n```\r\n\r\nWhile I already have these:\r\n```\r\n// Using FlowRouter\r\n    \"kadira:flow-router@2.0.2\",\r\n    \"kadira:blaze-layout@2.0.0\",\r\n```\r\n\r\nI think I'll try forking useraccounts:core and useraccounts:materialize and maybe remove the dependencies on iron router, then just try to play with it (Might just plan on using FlowRouter as is to define the routes)"}}