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

fix Bordeaux ENT #296

Merged
merged 4 commits into from
Mar 27, 2024
Merged

fix Bordeaux ENT #296

merged 4 commits into from
Mar 27, 2024

Conversation

tmonjalo
Copy link
Contributor

L'académie de Bordeaux utilise HubEduConnect.

The HubEduConnect method requires the Pronote URL to be passed.
This URL is passed when creating or refreshing the client instance.
In this commit, the URL is not used in any ENT function.
All ENT functions are made more flexible by accepting optional
keyword arguments (kwargs).
It was required to call pronote_hubeduconnect()
with the Pronote URL as argument when creating the client.
It is now moved with other ENTs,
getting the additional parameter "pronote_url" automatically.
The ENT ent2d_bordeaux was not working anymore.
It is replaced with the ENT "bordeaux" which uses HubEduConnect.

fixes bain3#272
fixes bain3#280
Several ENTs are processing SAML request/response.
Make it a single function used by the ENTs.
@bain3 bain3 merged commit e2390b4 into bain3:master Mar 27, 2024
3 of 4 checks passed
@bain3
Copy link
Owner

bain3 commented Mar 27, 2024

Thank you!

@tmonjalo tmonjalo deleted the bordeaux branch March 27, 2024 21:42
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.

2 participants