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

Connectors page: "refresh tokens" and "group membership" links incorrect #26

Open
nate-double-u opened this issue Oct 13, 2020 · 6 comments
Labels
question Further information is requested

Comments

@nate-double-u
Copy link
Contributor

In paragraph starting "Depending on the connectors limitations..." the refresh tokens and group membership links are broken.

Screen Shot 2020-10-13 at 3 18 49 PM

@nate-double-u
Copy link
Contributor Author

refresh tokens should link here: https://dexidp.io/docs/custom-scopes-claims-clients/#scopes

@nate-double-u
Copy link
Contributor Author

group membership should link here: https://dexidp.io/docs/custom-scopes-claims-clients/#scopes

@nate-double-u
Copy link
Contributor Author

@sagikazarmark, both these links are incorrectly linked, but looking at it closer they are both pointing to the same place -- when we change the link, should they still both link to: https://dexidp.io/docs/custom-scopes-claims-clients/#scopes?

@sagikazarmark
Copy link
Member

good question, I think some documentation is missing here :D

@nate-double-u
Copy link
Contributor Author

I've opened PR #37 for this to at least stops the links from 404ing.
We can have further discussion about where we would want these to link more permanently.

@nate-double-u nate-double-u removed their assignment Oct 15, 2020
@nate-double-u nate-double-u changed the title refresh tokens and group membership links broken on front page "refresh tokens" and "group membership" links broken on front page Oct 16, 2020
@nate-double-u nate-double-u changed the title "refresh tokens" and "group membership" links broken on front page Connectors page: "refresh tokens" and "group membership" links incorrect Oct 28, 2020
@nate-double-u
Copy link
Contributor Author

Updating issue title to reflect where this copy got moved to:
https://dexidp.io/docs/connectors/

@nate-double-u nate-double-u added the question Further information is requested label Oct 28, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

No branches or pull requests

2 participants