-
-
Notifications
You must be signed in to change notification settings - Fork 175
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
essec #173
essec #173
Conversation
ESSEC
@@ -412,6 +413,10 @@ | |||
"name": "ESC Clermont Business School", | |||
"AUTH_URL": "https://esc-clermont.idm.oclc.org/login?url=https://nouveau.europresse.com/access/ip/default.aspx?un=ESCT_1" | |||
}, | |||
{ | |||
"name": "ESSEC", | |||
"AUTH_URL": "https://nouveau-europresse-com.essec.idm.oclc.org/Search/Reading" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Je pense que cette url n'est pas correcte. Il faut faut qqch qui ressemble à ça:
https://essec.idm.oclc.org/login?url=https://nouveau.europresse.com/access/ip/default.aspx?un=UN_CODE_ICI
C'est l'url générale de connection à la plateforme qu'on a dû vous donner, il faut bien mettre l'url originale et pas la cible vers laquelle elle redirige.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Merci j'ai refait une pull request avec la bonne url
la nouvelle version n'est pas encore publiée sur Chrome |
Le processus est manuel, et il y a un délai pour l'acceptation par Google, donc j'attends d'avoir plusieurs changements pour les soumettre en même temps à Google. |
À propos des smartphones, vous pouvez lire cette discussion: #139 |
Et à propos de Safari: #131 |
Si quelqu'un veut faire un don de 100$/an pour couvrir les frais de submission, je suis prêt à faire le travail, et packager l'extension pour la distribuer sur safari. |
Ajouter ESSEC merci