Skip to content

Atualização asp.net core para 3.1#14

Open
robertluiz wants to merge 10 commits intoancapsu:masterfrom
robertluiz:master
Open

Atualização asp.net core para 3.1#14
robertluiz wants to merge 10 commits intoancapsu:masterfrom
robertluiz:master

Conversation

@robertluiz
Copy link

Fixes #9

Copy link
Contributor

@nullconst nullconst left a comment

Choose a reason for hiding this comment

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

Acho que o .gitignore acabou entrando em conflito, e tem alguns arquivos de projeto, que parecem transientes, em LibVisWeb/.vs.
Se importa de corrigir esses pontos?

De qualquer formar, como são alterações mais complexas, vou terminar de revisada e testar, mas vou deixar pro @peterturguniev decidir.

@robertluiz
Copy link
Author

robertluiz commented Jul 3, 2020

blz, acabou dando conflito por causa do pr anterior, vou corrigir, é bom mesmo dar uma boa testada mas do 2.1 para o 3.1 mudou mais a parte de inicialização, tive que dar um supress "UseWebpackDevMiddleware" que está descontinuado a partir dessa versão.
É bom ir pensando em uma nova abordagem para o frontend porque agora o asp.net não embuti mais nada do webpack. O bom seria deixar o projeto de react integralmente dentro da pasta "ClientApp" e usar tarefas e apontamentos dentro do asp.net, tem alguns boilerplates de exemplos aqui no github.
.NET Core / React / Redux / TypeScript - Starter Boilerplate

Copy link
Contributor

@nullconst nullconst left a comment

Choose a reason for hiding this comment

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

Testei superficialmente e parece estar funcionando. Mas ainda tem vários padrões repetidos ou desnecessários que foram adicionados no .gitignore, e arquivos, de cache ou de configurações transientes, adicionados no LibVisWeb/.vs/...

@robertluiz
Copy link
Author

Isso porque o repositório já estava com arquivos no .vs antes de conter a regra no .gitignore, esses arquivos precisão ser apagados aqui no repositório.

@robertluiz robertluiz requested a review from nullconst July 4, 2020 03:54
Copy link
Contributor

@nullconst nullconst left a comment

Choose a reason for hiding this comment

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

Tem razão, erro meu.
Estou aprovando pro @peterturguniev dar uma olhada.

@nullconst nullconst requested a review from peterturguniev July 4, 2020 05:19
@nullconst
Copy link
Contributor

@robertluiz, pra funcionar no docker, precisei alterar o Dockerfile:

diff --git a/Docker/Dockerfile b/Docker/Dockerfile
index d4defdd..2af6757 100644
--- a/Docker/Dockerfile
+++ b/Docker/Dockerfile
@@ -35,4 +35,4 @@ FROM mcr.microsoft.com/dotnet/core/sdk:3.1
 WORKDIR /app
 COPY --from=build /app .
 EXPOSE 80
-ENTRYPOINT ["dotnet", "LibVisWeb.dll"]
+ENTRYPOINT ["dotnet", "LibVisWeb.dll", "--urls", "http://0.0.0.0;https://0.0.0.0"]

Se quiser, pode incluir isso, senão eu faço depois que o @peterturguniev aprovar/fizer o merge.

@robertluiz
Copy link
Author

feito

@robertluiz robertluiz requested a review from nullconst July 10, 2020 16:29
@@ -100,17 +100,6 @@ $RECYCLE.BIN/
*.lnk

# End of https://www.toptal.com/developers/gitignore/api/windows,linux,osx

Choose a reason for hiding this comment

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

Esse comentário ainda faz sentido?

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.

Migrar LibVisLib de 2.1 para 3.1

3 participants