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.

Sign up for GitHub

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

Jump to bottom

Remove FOSUserBundle #23

Closed
roymiles opened this issue Aug 25, 2016 · 3 comments
Closed

Remove FOSUserBundle #23

roymiles opened this issue Aug 25, 2016 · 3 comments

Comments

@roymiles
Copy link
Owner

  • Going for custom. Too much limitations with FOSUserBundle (Plus Id instead of id :P)
  • Using Site references in all other tables with -1 as the local site
@roymiles
Copy link
Owner Author

What about the session system? Api Users will use tokens but local users will have session storage.

Simple. Have all GET,POST etc requests include a SiteId and if SiteId == -1 then check privileges from session token else check from access token in cookie

@roymiles roymiles modified the milestone: All in one user system Aug 25, 2016
@roymiles roymiles mentioned this issue Aug 25, 2016
Closed
@roymiles
Copy link
Owner Author

roymiles commented Aug 30, 2016 •
edited
Loading

Still need to remove the views

Do this once we've established the views for new custom system

@roymiles roymiles removed this from the All in one user system milestone Sep 10, 2016
@roymiles roymiles changed the title Remove FOSUserBundle (Final decision) Remove FOSUserBundle Oct 3, 2016
@roymiles
Copy link
Owner Author

roymiles commented Oct 3, 2016

Not using FOSUserBundle now and there are no dependencies on it. The view files are still there for reference.

@roymiles roymiles closed this as completed Oct 3, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@roymiles