diego3g / rsxp-2023

Repositório contendo o código do app do RS/XP 2023
MIT License
443 stars 85 forks source link

Pasta padrão do App mobile #83

Closed RennanD closed 1 year ago

RennanD commented 1 year ago

Acho melhor definirmos a utilização da pasta app como padrão para desenvolvimento da parte mobile. Como o Expo Router está funcionando lá e alguns componentes já foram criados lá, na minha opinião vale mais centralizar todos os outros recursos lá. 💜

diego3g commented 1 year ago

Podemos colocar tudo dentro da pasta app mesmo que não sejam rotas? Não manjo muito do Expo Router, mas no Next.js a gente evita colocar tudo na pasta app, deixamos lá somente pra arquivos relacionados às rotas da aplicação.

nitoba commented 1 year ago

@RennanD infelizmente isso não é possível, qualquer arquivo dentro da pasta app se torna automaticamente uma rota. IMG_20230406_184040

melquisedeque-magalhaes commented 1 year ago

ai fica a duvida atualmente na pasta app, nos temos uma pasta de componentes, @types, assets, fora as outras pastas que são realmente rotas da aplicação, essas outras pastas acho melhor ficar dentro da pasta src e deixar dentro de app, realmente so as rotas da aplicação, o mais interessante seria se pudéssemos colocar a pasta app dentro da pasta src mas não sei se e possível hoje com o expo router.

orodrigogo commented 1 year ago

@melquisedeque-magalhaes e @RennanD me parece ser possível definir a variável de ambiente no arquivo babel.config.js apontando o App dentro da pasta src com process.env.EXPO_ROUTER_APP_ROOT = __dirname + "/src/app";

Que tal essa tentativa?

melquisedeque-magalhaes commented 1 year ago

@melquisedeque-magalhaes e @RennanD me parece ser possível definir a variável de ambiente no arquivo babel.config.js apontando o App dentro da pasta src com process.env.EXPO_ROUTER_APP_ROOT = __dirname + "/src/app";

Que tal essa tentativa?

@rodrigorgtic Vou testar aqui para ver se vai kkk

melquisedeque-magalhaes commented 1 year ago

@rodrigorgtic mano deu ruim, não funcionou

Capturar_select-area_20230406225739

Capturar_select-area_20230406225754

nitoba commented 1 year ago

@melquisedeque-magalhaes tenta usar o 'require' ao invés do 'import' pra quando for arquivos js

melquisedeque-magalhaes commented 1 year ago

@melquisedeque-magalhaes tenta usar o 'require' ao invés do 'import' pra quando for arquivos js

usei mesmo assim não foi mano.

Capturar_select-area_20230406233625

nitoba commented 1 year ago

Estranho, pois esse erro da quando vc executa o app fora da pasta mobile

melquisedeque-magalhaes commented 1 year ago

Estranho, pois esse erro da quando vc executa o app fora da pasta mobile

erra isso mesmo mano, mas mesmo assim ele não pegou mano, ele não conseguiu pegar a pasta app dentro da pasta src

Capturar_select-area_20230406234243

orodrigogo commented 1 year ago

Galera, vou olhar aqui e já retorno pra vocês com a solução. 💜

vinifraga commented 1 year ago

Acho que podemos fechar essa issue @diego3g