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

BNF-only redirection mediapart #290

Merged
merged 2 commits into from
Feb 2, 2025

Conversation

JeremieGiffard
Copy link
Contributor

redirection vers l'article lié du mirroir mediapart

Autre solution pour verifier la connection au mirroir (la nuit porte conseil). Checker si le nom du compte BNF est present sur la page ou non

  • redirection
  • fetch de la page login si non connecté
  • suppression du setTimeout

@JohnCytron j'ai reçu ton message en mail, meme si j'arrive pas à le retrouver sur l'interface github ^^
Je test un peu plus, mais ça devrait regler ton problème de login. En incognito meme version de firefox, le login se faisait bien pour moi avant cette pr.

PR en draft le temps que je remette ça au propre. Et teste un peu.

@JohnCytron
Copy link

JohnCytron commented Feb 1, 2025

Quand j'ai fait le message, c'était le userscript https://github.com/snr-161/bnf-redirect qui faisait la redirection et qui posait le problème. J'aurais dû éditer mon message au lieu de le supprimer. Désolé pour le dérangement 😅.

Marche très bien maintenant, merci beaucoup pour le PR, c'est très utile.

@Write Write closed this Feb 1, 2025
@Write Write reopened this Feb 1, 2025
@Write
Copy link
Collaborator

Write commented Feb 1, 2025

Pour la compatibilité future avec d'autres bibliothèques, où le nom changerait pour chaque bibliothèque, je propose plutôt de vérifier si le bouton contient toujours le texte "Se connecter", car par exemple dans mon cas l'utilisateur est "BNUS".

@JohnCytron
Copy link

Ça serait peut être bien de mettre a jour le README pour les "Sites supportés" (en spécifiant la limitation à la BNF).

@JeremieGiffard
Copy link
Contributor Author

mon Prettier plugin qu'a fait un peu du zèle.

  • check si 'se connecter' present dans la navbar -> fetch login page (c'est en effet plus pratique pour plus tard ^^)
  • url du fetch dynamique

tests ok sur firefox 134.0.2 et chromium. En navigation standard et incognito.

Bon rappel, pour la liste des sites supportés. Je fais une pr à part. j'ai plusieurs sites bnf à rajouter.

@JeremieGiffard JeremieGiffard marked this pull request as ready for review February 2, 2025 10:21
@Write
Copy link
Collaborator

Write commented Feb 2, 2025

merci :)

@Write Write merged commit c837c8a into lovasoa:master Feb 2, 2025
1 check passed
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