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

Engade a sección "Os nosos amigos" e máis... #93

Merged
merged 6 commits into from
Apr 24, 2019

Conversation

sergiocarracedo
Copy link
Contributor

  • Engade a sección "os nosos amigos"
  • Recolle os amigos dende un json (friends.json)
  • Actualiza Nuxt a 2.5 e Vue a 2.6.1
  • Substitue asyncData polo novo prefechServer de Vue 2.6 para a xeneración de SSR
  • Oculta o html do popup das cookies que unha vez aceptado aparecia na parte de abaixo da web.
  • Actualiza o .env.example

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

Copy link
Member

@antonmry antonmry left a 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.

@sergiocarracedo
Copy link
Contributor Author

@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

@sergiocarracedo
Copy link
Contributor Author

Creei o issue #94 para debater sobre o procedemento de entrada. Comment please!

@antonmry
Copy link
Member

antonmry commented Apr 2, 2019

Polo meu lado está perfecto. Ademáis da issue, habería que avisar na lista de correo, é a canle oficial

@sergiocarracedo
Copy link
Contributor Author

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.

@antonmry
Copy link
Member

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

@sergiocarracedo
Copy link
Contributor Author

@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

@sergiocarracedo
Copy link
Contributor Author

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

@antonmry
Copy link
Member

por min facemos merge xa desta feature, meter os amigos pode ser unha PR diferente, non?

@sergiocarracedo
Copy link
Contributor Author

ok, si agora mesmo o json de "friends" esta valeiro.

@sergiocarracedo sergiocarracedo merged commit f667b60 into VigoTech:source Apr 24, 2019
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.

2 participants