search

Scribouilli / scribouilli

https://atelier.scribouilli.org/
MIT License
73 stars 14 forks source link

:ant: Je peux utiliser Scribouilli sur plusieurs repo 👍👍 #29

Closed Ynote closed 1 year ago

Ynote commented 1 year ago

Scénarii pour la création :

Il faut également penser à une page pour permettre à l'utilisateurice de changer de repo.

maiwann commented 1 year ago
maiwann commented 1 year ago

Résolu si on ne met pas d'espaces dans l'input ! image

Note : en fait les 2 dépots ont été créés, mais celui avec des espaces a déclenché une erreur 🤔

image

Du coup est-ce que ça affiche une fausse erreur ? Est-ce qu'on veut décourager les gens de mettre des espaces ? Est-ce qu'il faut qu'on gère les espaces d'une façon différente ?

Note bis : J'ai reçu un mail de Github qui a l'air de dire que qqchose va pas alors que ma modif (de style) a fonctionné ? image

maiwann commented 1 year ago

[en cours d'investigation]

Mail reçu à l'instant :

Bonjour Maïtané, J'ai tout recommencé en prenant en compte tes indications ici : https://chloethorey.github.io/test-website-repo-3796/ pour que les pages soient dans le bon ordre. Mais aujourd'hui, comme j'avais du temps pour compléter la dernière page et tenter une mise en ligne, je me suis reconnectée via un marque page que je m'étais mis sur scribouilli et je ne retombe pas sur mon projet, mais sur le site : https://scribouilli.github.io/scribouilli/atelier-list-pages?repoName=test-website-repo-3796&account=[object%20Object] Est ce qu'il existe une possibilité pour que je retrouve mon projet en cours?

elegaanz commented 1 year ago

Ce lien a l'air de marcher (en remplaçant le [object Object] dans l'URL par son nom de compte) : https://scribouilli.github.io/scribouilli/atelier-list-pages?repoName=test-website-repo-3796&account=chloethorey

(d'ailleurs il marche pour tout le monde, je sais pas si c'est un problème ou pas, en pratique je ne pense pas qu'on puisse éditer son site, mais ça donne cette impression, et je ne sais pas si c'est un souci ou pas)

Ynote commented 1 year ago

Mais aujourd'hui, comme j'avais du temps pour compléter la dernière page et tenter une mise en ligne, je me suis reconnectée via un marque page que je m'étais mis sur scribouilli et je ne retombe pas sur mon projet, mais sur le site : https://scribouilli.github.io/scribouilli/atelier-list-pages?repoName=test-website-repo-3796&account=[object%20Object]

Je ne suis pas certaine de voir exactement à quel moment la redirection fait ça. Je creuse un peu.

d'ailleurs il marche pour tout le monde, je sais pas si c'est un problème ou pas, en pratique je ne pense pas qu'on puisse éditer son site, mais ça donne cette impression, et je ne sais pas si c'est un souci ou pas

Oui c'est embêtant ça qu'on puisse toustes y accéder, je vais jeter un oeil.

Ynote commented 1 year ago

Du coup est-ce que ça affiche une fausse erreur ? Est-ce qu'on veut décourager les gens de mettre des espaces ? Est-ce qu'il faut qu'on gère les espaces d'une façon différente ?

@maiwann Je propose pour nous faciliter la vie de décourager les espaces. Plusieurs possiblités (qu'on peut combiner ou pas):

Ma préférence c'est la 3ème, je crois que les utilisateurices ne vont pas souvent éditer leur repo et vont à peine se rendre compte que les espaces ont été changé en tiret. Mais ça m'embête de faire des trucs sans l'accord des utilisateurices. On peut en discuter de vive voix pour voir ce qu'on choisit de faire.

Ynote commented 1 year ago

Je viens de tester et je vois que GitHub transforme les espaces en tiret en fait. On pourrait donc faire pareil.

Ynote commented 1 year ago

d'ailleurs il marche pour tout le monde, je sais pas si c'est un problème ou pas, en pratique je ne pense pas qu'on puisse éditer son site, mais ça donne cette impression, et je ne sais pas si c'est un souci ou pas

J'ai pris le parti pour l'instant de rediriger si on n'est pas owner du repo (cf. https://github.com/Scribouilli/scribouilli/pull/39). Je me permets de merger cette PR car je pense que ce sera nécessaire dans tous les cas de faire ce check lorsqu'on ajoutera les organisations.

Ynote commented 1 year ago

Mais aujourd'hui, comme j'avais du temps pour compléter la dernière page et tenter une mise en ligne, je me suis reconnectée via un marque page que je m'étais mis sur scribouilli et je ne retombe pas sur mon projet, mais sur le site : https://scribouilli.github.io/scribouilli/atelier-list-pages?repoName=test-website-repo-3796&account=[object%20Object] Est ce qu'il existe une possibilité pour que je retrouve mon projet en cours?

J'ai trouvé 🎉 C'est fixé avec https://github.com/Scribouilli/scribouilli/pull/40.

Ynote commented 1 year ago

@maiwann Sur les bugs listés dans cette issue, il reste ça pour l'instant

Note bis : J'ai reçu un mail de Github qui a l'air de dire que qqchose va pas alors que ma modif (de style) a fonctionné ?

Je m'en occupe en fin de semaine si c'est ok pour toi. Ou si d'autres personnes sont motivées :D

Ynote commented 1 year ago

Du coup est-ce que ça affiche une fausse erreur ? Est-ce qu'on veut décourager les gens de mettre des espaces ? Est-ce qu'il faut qu'on gère les espaces d'une façon différente ?

Say bon \o/ Avec #42, on peut créer sans erreur des sites avec des espaces et des caractères spéciaux. Je clean tout côté back avant d'envoyer le call à l'API Github.

Ynote commented 1 year ago

Note bis : J'ai reçu un mail de Github qui a l'air de dire que qqchose va pas alors que ma modif (de style) a fonctionné ?

@maiwann Après petit enquête 🔍, c'est normal de recevoir ce mails transactionnel si tu fais plusieurs modifs d'affilée sur différentes pages ou styles. GitHub Actions annule automatiquement les déploiements s'il y en a un nouveau en cours.

Pour gérer les emails transactionnels de GitHub, il y a cette documentation qu'on pourrait ajouter dans notre documentation également. Je n'ai pas trouvé de moyen pour gérer ça directement via l'API GitHub.

maiwann commented 1 year ago

Merci @ynote pour tous ces fix <3