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

DataPass API - Lecture / écriture #447

Open
wants to merge 11 commits into
base: develop
Choose a base branch
from
Open

DataPass API - Lecture / écriture #447

wants to merge 11 commits into from

Conversation

skelz0r
Copy link
Member

@skelz0r skelz0r commented Sep 23, 2024

No description provided.

@JeSuisUnCaillou
Copy link
Contributor

Il nous manque quoi pour que ce ne soit plus un draft ? Des specs ? Du café ? :D

@skelz0r
Copy link
Member Author

skelz0r commented Dec 19, 2024

Un utilisateur. API E/P va très certainement arriver d'ici quelques semaines.

@skelz0r
Copy link
Member Author

skelz0r commented Dec 19, 2024

C'est dérangeant que ça reste en draft ?

@JeSuisUnCaillou
Copy link
Contributor

Non non, je me demandais juste pourquoi on l'avait pas merge

Doorkeeper is an OAuth2 server gem. More info: https://github.com/doorkeeper-gem/doorkeeper

Custom config:

* fr i18n
* inherits from APIController (make some iterations to be compliant)
* api mode only -> no need to manage application with default UI (maybe
  will be some UI in the future)
* skip controllers within routes related to UI (related to previous
  point)
* each application has an owner (https://github.com/doorkeeper-gem/doorkeeper/wiki/Associate-users-to-OAuth-applications-%28ownership%29)
* configure scopes
Dummy /me, without serializers.
Will need some refactor for testing.
Use redoc to make OpenAPI file available on the site
We can have old/invalid key within data, just ignored them
@skelz0r skelz0r marked this pull request as ready for review January 20, 2025 16:24
@skelz0r
Copy link
Member Author

skelz0r commented Jan 20, 2025

allez hop() au moins c'est ship

@skelz0r
Copy link
Member Author

skelz0r commented Jan 20, 2025

A noter que pour le moment il n'y a aucun outil pour récupérer ses creds d'app OAuth2 en tant que developer, et rien du tout pour créer non plus.

Copy link
Contributor

@Samuelfaure Samuelfaure left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

very good

Tu as utilisé un outil pour générer le OpenAPI ?

app/controllers/concerns/authentication.rb Show resolved Hide resolved
@skelz0r
Copy link
Member Author

skelz0r commented Jan 21, 2025

Tu as utilisé un outil pour générer le OpenAPI ?

https://chatgpt.com/share/678f704d-2558-8008-ab54-b9a6253807c8

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