You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I read the readme and I can appreicate the intent to make something that can be resued and extended. But right now, the coupling is tight and specific to foxbox and the extra layer of indirection doesn't help us either.
Making a iron-user-module crate is a different goal than building Project Link and I don't think we should sign up for that. I've created a few user systems before 😉 and based on experience, this stuff is better as a boilerplate than of a library/crate.
With that said, I feel strongly that we should merge this repo with foxbox. There should only be 5-6 files to move over into a module directory. I can help doing this.
Although I agree that this is far from being the core goal of Project Link, I still think that there is value on keeping this library external. The fact that it needs to evolve along with Project Link's development and requirements does not mean that the code needs to be specific to it. If things goes the way that they are planned, we will end up with a generic multi-user authentication and authorization system. We need to build it for FoxBox anyway, so why not allow others to reuse our work? IMHO the development cost of having this code external to FoxBox has been quite minimal so far. Could you please point to specific issues that having this code as a library has caused so far?
In any case, I'll defer to @fabricedesre to make the call.
I read the readme and I can appreicate the intent to make something that can be resued and extended. But right now, the coupling is tight and specific to foxbox and the extra layer of indirection doesn't help us either.
Making a iron-user-module crate is a different goal than building Project Link and I don't think we should sign up for that. I've created a few user systems before 😉 and based on experience, this stuff is better as a boilerplate than of a library/crate.
With that said, I feel strongly that we should merge this repo with foxbox. There should only be 5-6 files to move over into a module directory. I can help doing this.
📟 @fabricedesre @ferjm @arcturus
The text was updated successfully, but these errors were encountered: