Skip to content
This repository has been archived by the owner on Jan 6, 2023. It is now read-only.

Add referral URL in request_token #127

Closed
rijkvanzanten opened this issue May 11, 2018 · 3 comments
Closed

Add referral URL in request_token #127

rijkvanzanten opened this issue May 11, 2018 · 3 comments
Labels
enhancement New feature or request

Comments

@rijkvanzanten
Copy link
Member

When the API redirects to the app, the following payload is provided in the token:

{
  "type": "request_token",
  "id": 4,
  "exp": 1526061779
}

However, there is no way of knowing what API to log into with this..

To fix this, we should add the API url to this payload.

{
  "type": "request_token",
  "id": 4,
  "exp": 1526061779,
  "url": "https://demo-api.getdirectus.com"
}
@rijkvanzanten rijkvanzanten added the bug Something isn't working label May 11, 2018
@rijkvanzanten rijkvanzanten added this to To do in v2.0.0-alpha.3 via automation May 11, 2018
@wellingguzman wellingguzman added enhancement New feature or request and removed bug Something isn't working labels May 11, 2018
v2.0.0-alpha.3 automation moved this from To do to Done May 11, 2018
@rijkvanzanten
Copy link
Member Author

@wellingguzman Just realized we also need the env in use!

(The app doesn't allow selecting an env yet, but will eventually)

Lets extend this from

{
  "type": "request_token",
  "id": 4,
  "exp": 1526065812,
  "url": "http://localhost/api/public/"
}

to

{
  "type": "request_token",
  "id": 4,
  "exp": 1526065812,
  "url": "http://localhost/api/public/",
  "env": "_"
}

@rijkvanzanten rijkvanzanten reopened this May 11, 2018
v2.0.0-alpha.3 automation moved this from Done to In progress May 11, 2018
wellingguzman added a commit that referenced this issue May 11, 2018
@wellingguzman
Copy link
Contributor

wellingguzman commented May 11, 2018

I added the env here: 5e1ccaf

@rijkvanzanten
Copy link
Member Author

Purrrrrrrfect 🐈

v2.0.0-alpha.3 automation moved this from In progress to Done May 11, 2018
samvasko pushed a commit to samvasko/api that referenced this issue Nov 25, 2019
* Upgrade packages

* Move bookmark name to right of breadcrumb

Ref directus#131

* Change android pwa standalone color

Ref directus#33

* Use CSS hover for enabling / disabling user menu

Ref directus#33

* Tweak latency times

Ref directus#93

* Change logout 'expired' to 'inactivity'

Closes directus#100

* Debounce filter input

Closes directus#127

* Re-add borders in user menu

Closes directus#130

* Remove reference to usermenuactive state

* Only open menu when user is tabbing and focus

* Breadcrumb bookmark styling

Closes directus#131

* Handle gravatar error better

* Render hydration error (needs styling)

* Only start polling after successful hydration

* Rework icon placement styling

Ref directus#132

* Style delete bookmark button, and show on hover

Ref directus#132

* Don't wrap bookmark names

Ref directus#132

* Update spacing of nav items in sidebar'

* Tweak spacing of nav headers

* Dont wrap bookmark name in header bar

* Add modal for sign out

* Add sign out confirm modal functionality

Closes directus#133

* Support rendering readonly in v-table

* Add some various translations

* Dont allow overflowing in table cells

* Add file key to en-US locales

* Change yes to sign out in sign out modal
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
No open projects
Development

No branches or pull requests

2 participants