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

[Omniauth] Changement de stratégie d'autoload sur la gem #5036

Open
wants to merge 2 commits into
base: production
Choose a base branch
from

Conversation

Michaelvilleneuve
Copy link
Collaborator

L'autoload des fichiers de la gem ne fonctionne pas une fois émise sur rubygems. Je change légèrement la stratégie, j'ai eu du succès avec celle-ci dans le passé.

@victormours
Copy link
Contributor

Je suis pas sûr de comprendre la description : est-ce que tu es sûr que ça marche avec ce code ou est-ce que c'est une tentative ? Si c'est le deuxième cas il vaut mieux modifier les fichiers en local (avec un bundle show omniauth-rdv-service-public) pour valider ça.

@Michaelvilleneuve
Copy link
Collaborator Author

Je suis pas sûr de comprendre la description : est-ce que tu es sûr que ça marche avec ce code ou est-ce que c'est une tentative ? Si c'est le deuxième cas il vaut mieux modifier les fichiers en local (avec un bundle show omniauth-rdv-service-public) pour valider ça.

@victormours En fait la Gem qui est actuellement sur Rubygems est vraiment cassée, les fichiers n'apparaissent même pas.

image

C'est pour ça que je suggérais cette modification en attendant d'avoir accès et de pouvoir publier une version pour faire un essai.
Pourras-tu m'ajouter comme collaborateur please 🙏 MichaelBeta

@victormours
Copy link
Contributor

Ah ok je comprends mieux. C'est donc surtout l'ajout de gem.files qui fait qu'on ajoute les fichiers correctement. Je me rends compte qu'on a le même problème sur la gem de l'anonymizer, je vais faire un autre fix pour ça.
Normalement tu as reçu une invitation à être maintainer sur la gem (ruby gems me dit qu'elle est "pending" pour MichaelBeta).

Copy link
Contributor

@victormours victormours left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 merci pour le correctif !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: 🚀 To deploy
Development

Successfully merging this pull request may close these issues.

2 participants