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 : accès à pressreader #287

Merged
merged 1 commit into from
Jan 27, 2025

Conversation

JeremieGiffard
Copy link
Contributor

BNF developpement specifique

#22

Ajout du site pressreader pour les abonné.e.s BNF .
Principe du site : version numérisé, de nombreuses dernière ou avant-dernière edition de journaux qui sortent en kiosque.

(Je galerai sur la redirection mediapart, du coup me suis dit, je fais rapidement, un autre plus facile. erf, ça s'est pas exactement passé comme prevu.)

pressReader a une navigation entièrement reactive sans reload de page. J'ai du selectionner la balise <body> , et mettre le bouton en fixed tout en haut de la page. Il saute sinon, avec toutes les modifications du DOM faites par le framework js du site.

Lien BNF prend en charge la redirection, via la première url lorsqu'on arrive sur le site. cependant, le lien ne sera pas update lors de la navigation (js based).
exemple :

  • je click sur un lien d'un moteur de recherche https://www.pressreader.com/fr/catalog
    • bouton 'lire avec BNF' redirigera vers le miroir/fr/catalog
  • je navigue sur le site https://www.pressreader.com/foryou
    • bouton bouton 'lire avec BNF' redirigera toujours vers le miroir/fr/catalog

Pas vraiment un problème, vu la manière de naviguer sur le site une fois connecté.

	new file:   ophirofox/content_scripts/pressreader.css
	new file:   ophirofox/content_scripts/pressreader.js
	modified:   ophirofox/manifest.json
@JeremieGiffard JeremieGiffard marked this pull request as ready for review January 26, 2025 21:44
@merwok
Copy link

merwok commented Jan 27, 2025

Est-ce que le système de config actuel permettrait aussi d’ajouter la connexion à pressreader via BaNQ (biblio et archives nationales du Québec)?

J’ai vu la discussion pour ajouter mediapart et autres qui sont spécifiques à BNF, et ça me donne envie de contribuer pour ajouter les services offerts par BaNQ!

@Write
Copy link
Collaborator

Write commented Jan 27, 2025

C'est un bon premier jet, si quelqu'un de motivé vient par là, il pourra toujours refaire un PR avec un MutationObserver, donc pas grave, c'est comme ça.
C'est déjà bien si ça fonctionne.

Je merge en attendant, je ne peux pas tester de toute façon car je n'ai pas d'identifiant.

Est-ce que le système de config actuel permettrait aussi d’ajouter la connexion à pressreader via BaNQ (biblio et archives nationales du Québec)?

J’ai vu la discussion pour ajouter mediapart et autres qui sont spécifiques à BNF, et ça me donne envie de contribuer pour ajouter les services offerts par BaNQ!

Oui j'imagine que oui !

@Write Write merged commit b12416b into lovasoa:master Jan 27, 2025
1 check passed
@Write
Copy link
Collaborator

Write commented Jan 28, 2025

Est-ce que le système de config actuel permettrait aussi d’ajouter la connexion à pressreader via BaNQ (biblio et archives nationales du Québec)?

J’ai vu la discussion pour ajouter mediapart et autres qui sont spécifiques à BNF, et ça me donne envie de contribuer pour ajouter les services offerts par BaNQ!

Je pense que j'ai parlé trop vite.

Pour le moment la BNF est codée en dur dans pressreader.js :

const config = await configurationsSpecifiques(['BNF'])

Mais il suffira de faire une migration pour obtenir l'Université/Bibliothèque choisie actuellement dans le code config.js qui vérifiera si la clé AUTH_URL_PRESSREADER existe pour celle sélectionnée actuellement.

@JeremieGiffard JeremieGiffard deleted the feature/bnf-pressreader branch January 29, 2025 20:45
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