-
Notifications
You must be signed in to change notification settings - Fork 20
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
Engade a sección "Os nosos amigos" e máis... #93
Conversation
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.
Este non é un cambio trivial, non deberíase publicar un listado de amigos sen consenso na lista de organizadores, igual que non se debería modificar o json de grupos sen esa validación.
En caso de consenso, deberíase documentar o procedemento.
@antonmry completamente dacordo, non é un tema "programatico", é mais de política. Se queredes podo valeirar o friends.json e asi a sección ocultase ate que se engadan os definitivos noutro PR |
Creei o issue #94 para debater sobre o procedemento de entrada. Comment please! |
Polo meu lado está perfecto. Ademáis da issue, habería que avisar na lista de correo, é a canle oficial |
Engadín un novo commit a PR. Partindo da idea de que en CorunaDevelopers fixeron un fork de este repo coa intención de que servise como base para outros metagrupos, fixen algo que tiña pensado facer fai tempo que é crear una ferramenta independente para validar e parsear o vigotech.json, esta ferramenta é a que se encarga de obter os eventos de Meetup, Eventbrite, json ou os vídeos de Youtube ou Teltek. A idea de facer unha ferramenta independente é que outras comunidades poidan aportar os seus importadores (por exemplo unha que teña os seus videos en Vimeo) e facelo sen necesidade de forkear o repo completo. Outra vantaxe de ter esta ferramenta separada é que se pode empregar en sideprojects da comunidade, como o https://github.com/VigoTech/vigotech-event-bot que agora esta tirando o json xerado pola web, pero que debería facelo por si solo para ter maior flexibilidade. Polo momento o repo esta na miña conta: https://github.com/sergiocarracedo/metagroup-schema-tools pero non teño problema en pasarllo a de Vigotech, se esta idea vos parece ben Para maior facilidade de uso tamén a subín como paquete a NPM https://www.npmjs.com/package/metagroup-schema-tools Espero os vosos comentarios. |
A mín paréceme moi boa idea, seguro que CoruñaDevelopers o agradece. O de pasarlo a conta de VigoTech é boa idea por ter todo no mesmo sitio |
@antonmry ok, fago o cambio de "titularidade" do repo e vexo como facer o mesmo co npm, e cando o teña listo actualizao o PR |
Transferido o repo e modificado o package.json VigoTech/metagroup-schema-tools#1 Para NPMjs non vin a forma de transferir o paquete pero engadín o usuario vigotech para que poida modificalo |
por min facemos merge xa desta feature, meter os amigos pode ser unha PR diferente, non? |
ok, si agora mesmo o json de "friends" esta valeiro. |
Antes de merguear deberiamos decidir a lista de amigos a engadir a web, eu engadin a lista que comentou @orestes no slack, pero podemos completala ou modificala