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

Add Sorbonne Université #77

Merged
merged 5 commits into from
Sep 1, 2023
Merged

Add Sorbonne Université #77

merged 5 commits into from
Sep 1, 2023

Conversation

AkaiDown
Copy link
Contributor

Addition of Sorbonne Université

AkaiDown and others added 2 commits August 28, 2023 17:39
Addition of Sorbonne Université
ophirofox/manifest.json Outdated Show resolved Hide resolved
@lovasoa
Copy link
Owner

lovasoa commented Aug 28, 2023

Une version de test de l'extension a été construite avec les changements que vous proposez.

Pour tester l'extension, il faut décompresser le fichier zip joint, et charger le fichier qui est à l'intérieur (qui est un autre zip) comme une extension (sur firefox, dans about:debugging#/runtime/this-firefox, en cliquant sur "charger une extension temporaire").

image

image

image

Ensuite il faut sélectionner votre université dans les préférences de l'extension

image

puis ouvrir un article payant, par exemple https://www.lemonde.fr/planete/article/2023/08/18/une-canicule-tardive-s-installe-en-france-symptome-du-rechauffement-climatique_6185847_3244.html

image

Et cliquer sur le lien "lire sur europresse".

@AkaiDown
Copy link
Contributor Author

Bon, ça ne fonctionne effectivement pas... Je vais chercher si je peux trouver un lien susceptible de marcher.

@lovasoa lovasoa changed the base branch from master to v2.3 August 29, 2023 07:27
@angedurand
Copy link

angedurand commented Sep 1, 2023

Bon, ça ne fonctionne effectivement pas... Je vais chercher si je peux trouver un lien susceptible de marcher.

Bonjour,

je crois que l'URL à utiliser est

https://accesdistant.sorbonne-universite.fr/login?url=https://nouveau.europresse.com/access/ip/default.aspx?un=UPMCT_1

EDIT: J'ai fait le test chez moi, ça fonctionne avec cette URL. Par contre je ne sais pas comment effectuer la correction ici.

@lovasoa
Copy link
Owner

lovasoa commented Sep 1, 2023

Super merci, j'ai fait le changement. Si vous confirmez que l'extension construite ici fonctionne, je l'intègre.

@AkaiDown
Copy link
Contributor Author

AkaiDown commented Sep 1, 2023

Super, ça semble fonctionner!

@lovasoa lovasoa merged commit 197719c into lovasoa:v2.3 Sep 1, 2023
1 check passed
@lovasoa lovasoa mentioned this pull request Sep 1, 2023
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