Skip to content

Improve the authentication documentation #535

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

Merged
merged 2 commits into from
Apr 22, 2025
Merged

Conversation

nirvn
Copy link
Member

@nirvn nirvn commented Mar 8, 2025

Improvements include:

  • moving away from being an OAuth2-specific page; we support all authentication methods and the current page makes it hard to understand we do
  • removing a couple of "she" usages in favor of "you".
  • removing mention of QGIS 3.20, as minimum version, we're many years into having moved away from that version, we can lighten that part of the documentation

@beanzmo
Copy link
Contributor

beanzmo commented Apr 16, 2025

@nirvn I would suggest to have a video here. This is not so easy. And perhaps, we could give one or two examples of where this would be useful to implement?

@nirvn
Copy link
Member Author

nirvn commented Apr 19, 2025

@beanzmo , could we merge this as is, and eventually add a video? It'll be hard to make one to begin with, we need a SSO WFS/WMS server, etc.

@nirvn nirvn force-pushed the authentication_imp branch from 6759b1b to a160df7 Compare April 19, 2025 12:43
@nirvn
Copy link
Member Author

nirvn commented Apr 19, 2025

@m-kuhn , thank(s) 😉 I went way overboard with these (s).

@beanzmo beanzmo merged commit d0cf8b6 into master Apr 22, 2025
3 checks passed
@beanzmo beanzmo deleted the authentication_imp branch April 22, 2025 11:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants