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

Regroup API Signin Routes + Transfer UI conditions to API #60

Closed
fmigneault opened this issue Jun 8, 2018 · 2 comments · Fixed by #119
Closed

Regroup API Signin Routes + Transfer UI conditions to API #60

fmigneault opened this issue Jun 8, 2018 · 2 comments · Fixed by #119
Assignees
Labels
enhancement Improvements in term of performance or behaviour WIP

Comments

@fmigneault
Copy link
Collaborator

fmigneault commented Jun 8, 2018

TODO:

  • make a single route /singin for any type (local ziggurat or external provider)
    (handling of external redirects to be done by UI)
  • let the API do the provider type switch, not the UI
  • update REST API docs in repo + swaggerhub

Checks:

  • ensure that external signin providers work (redirect to their login page + logged in after)
  • ensure that cookies are properly saved/cleared from sigin/signout via API & UI
  • ensure JSON & form/data compatibility for /signin (ref API requests with POST not accepting JSON format #50)

References / track source:

Extra:

@fmigneault fmigneault added the enhancement Improvements in term of performance or behaviour label Jun 8, 2018
@fmigneault fmigneault self-assigned this Jun 8, 2018
@fmigneault
Copy link
Collaborator Author

image

@fmigneault fmigneault added the WIP label Jun 26, 2018
@fmigneault
Copy link
Collaborator Author

fmigneault commented Aug 15, 2018

Relates to issue #68 and OGC-20

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Improvements in term of performance or behaviour WIP
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant