alteracoes, grupo 5 by giovaner10 · Pull Request #1 · renato3x/devgram

giovaner10 · 2022-03-27T23:35:24Z

Alterações na estrutura do projeto, integrando a responsividade e a utilização de rotas

renato3x

Há muitas incongruências no seu código, como falta de organização no desenvolvimento e também vocês fugiram do escopo de que isso é um projeto pessoal.

renato3x · 2022-03-28T20:09:42Z

src/app/components/perfil/perfil.component.html

@@ -0,0 +1 @@
+<p>a pagina de perfil vai ficar bem aqui</p>


Qual o sentido de criar uma nova página e não implementá-la?

Ela ainda esta em construção , só reservamos o espaço para a sua implantação.

renato3x · 2022-03-28T20:11:14Z

src/app/components/navbar/navbar.component.html

  <div class="img-logo">
    <a href="#">
-      <img [src]="instagramLogoSource" alt="Instagram - Logo">
+      <img [src]="instagramLogoSource" alt="Instagram - Logo" routerLink="">


Qual o motivo de colocar um routerLink vazio e ainda dentro de um elemento de imagem?

No elemento externo a imagem, o link não estava funcionando, o vazio se dar para desviar para rota de home, deixei uma path vazia, mas antes estava "path: 'home'", acabei esquecendo de remover. De toda forma era desnecessário, ja que o "href='#'", ia direcionar para o home

renato3x · 2022-03-28T20:12:53Z

src/app/components/aside-data/aside-data.component.css

@@ -1,3 +1,13 @@
+@media(max-width: 900px){


código da responsividade mal organizado

Vou concertar, irei criar uma folha de estilo exclusiva para as mesmas.

renato3x · 2022-03-28T20:13:46Z

src/app/components/aside-data/aside-data.component.html

    <div class="user-name-infos">
-      <span class="user-username">Renato3x</span>
-      <span class="user-name">Renato</span>
+      <span class="user-username">SoulCodeAcademy</span>


Isso continua sendo um projeto pessoal, de portifólio. Alterar isso não é aceito

Desculpa professor, não sabia, achei que íamos somente alterar e apresentar.

renato3x · 2022-03-28T20:16:02Z

src/app/app-routing.module.ts

+
+  {
+    path: 'soulcodeacademy',
+    component: PerfilComponent


Pelo que vi, esse componente PerfilComponent é um componente que é uma página. Neste projeto, há uma pasta exclusivamente para isso. Criar em outra pasta, afeta a organização do código. Além disso, note que foi utilizado o padrão inglês no desenolvimento. Fugir dessa organização pode ser ruim

Tudo bem professor, isso se deu porque ainda não mexemos nessa parte da nova pagina, já que a responsabildade da criação dela era de outros integrantes do grupo, e ela ainda esta em desenvolvimento.

renato3x · 2022-03-28T20:25:49Z

Notei que vocês cometeram um bug na parte dos stories: as alterações que vocês fizeram, fizeram com que não fosse mais possível navegar entre os stories, ou seja, não consigo mais fazer a rolagem entre os stories.

giovaner10 · 2022-03-28T23:18:29Z

Professor, em questão ao problema da rolagem entre os stories, eu acabei de pegar no projeto original, e também não há rolagem entre os stories, clico e não acontece nada.

…tanto em proporcoes para desketop, quanto para responsive

tikaoo · 2022-04-04T16:25:23Z

Professor, incluí rotas "filhas" para aplicação do lazy loading está correto dessa forma? Lembrando que ainda falta a elaboração de uma página de perfil que será criada.

renato3x · 2022-04-05T16:48:03Z

src/app/modulos/component-routing.module.ts

+
+const routes: Routes = [
+{
+  path:'aside-data',


Qual a necessidade de criar rotas para mostrar um componente de cada vez?

renato3x · 2022-04-05T16:58:57Z

src/app/modulos/perfil-routing.module.ts

+import { RouterModule, Routes } from '@angular/router';
+import { PerfilComponent } from '../components/perfil/perfil.component';
+
+const routes: Routes = [


Por qual motivo fazer um módulo de roteamento unicamente para a página de perfil? A aplicação é simples, considero que não seria necessário fazer isso. Além disso, ainda notei que a página de perfil ainda está no lugar errado. Se o componente Perfil é uma página, ele deveria estar na pasta /pages, junto com o componente Home. E também ele ainda não segue o padrão inglês

renato3x · 2022-04-05T17:00:17Z

src/app/app-routing.module.ts

+
+  {
+    path: 'soulcodeacademy',
+    loadChildren: ()=> import('./components/perfil/perfil.module').then(m=> m.PerfilModule)


É uma boa ideia a do Lazy Loading. Mas não estou vendo necessidade de utilizá-lo nesse projeto

renato3x · 2022-04-05T17:05:18Z

Vocês deixaram os módulos adicionais que vocês criaram totalmente desorganizados. O dois módulos do módulo component (o principal e o routing) estão em lugares diferentes. Isso, na organização de um projeto Angular real, é totalmente errado. Aconselho a desfazerem estas últimas alerações

…ava muito para o carregamento, pois estaticamente ja carregava 50 publicações, com a adoção do scrool, ela carrega estaticamente 5 posts, e dinaminicamente ela carrega os demais a mediade que rolamos a pagina. Usei a ngx-infinite-scroll para esse implementação

renato3x · 2022-04-11T13:27:32Z

Por favor, me informe o nome de todos os integrantes

…o grupo5

This reverts commit f80c828.

Bumps [minimist](https://github.com/substack/minimist) from 1.2.5 to 1.2.6. - [Release notes](https://github.com/substack/minimist/releases) - [Commits](https://github.com/substack/minimist/compare/1.2.5...1.2.6) --- updated-dependencies: - dependency-name: minimist dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com>

…mist-1.2.6 Bump minimist from 1.2.5 to 1.2.6

Bumps [eventsource](https://github.com/EventSource/eventsource) from 1.1.0 to 1.1.1. - [Release notes](https://github.com/EventSource/eventsource/releases) - [Changelog](https://github.com/EventSource/eventsource/blob/master/HISTORY.md) - [Commits](EventSource/eventsource@v1.1.0...v1.1.1) --- updated-dependencies: - dependency-name: eventsource dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com>

Bumps [async](https://github.com/caolan/async) from 2.6.3 to 2.6.4. - [Release notes](https://github.com/caolan/async/releases) - [Changelog](https://github.com/caolan/async/blob/v2.6.4/CHANGELOG.md) - [Commits](caolan/async@v2.6.3...v2.6.4) --- updated-dependencies: - dependency-name: async dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com>

…c-2.6.4 Bump async from 2.6.3 to 2.6.4

…tsource-1.1.1 Bump eventsource from 1.1.0 to 1.1.1

giovaner10 added 3 commits March 27, 2022 20:29

alteracoes, grupo 5

918a078

novas atualizacoes

339cb9d

alteracoes no responsive nav bar

8515f14

renato3x added the bug Something isn't working label Mar 28, 2022

renato3x reviewed Mar 28, 2022

View reviewed changes

tikaoo and others added 2 commits April 2, 2022 20:42

Adicionando Lazy Loading

bd07871

mudancas no html do home component visando adequacao ao design real, …

74fb9d0

…tanto em proporcoes para desketop, quanto para responsive

renato3x removed the bug Something isn't working label Apr 5, 2022

renato3x reviewed Apr 5, 2022

View reviewed changes

giovaner10 added 3 commits April 5, 2022 17:57

adicao do router link para o perfil no asside

e41bfbf

colocao do pointer nos itens que vao direcionar para perfil no asside

9dbe61f

IsisFer and others added 13 commits April 11, 2022 20:26

inserindo footer

3d764c9

revertendo lazy loading

9042304

revertendo lazy loading

4ac87ae

Mid section do perfil principal

3147b02

Arrumando responsividade das imagens Mid section

9ebd412

Fixing again :(

3427cac

top-section do perfil principal

84fa0e2

update

839f0c1

update

a9dc463

update

f80c828

Merge branch 'grupo5' of https://github.com/alvaroaxsmith/devgram int…

c55f4c8

…o grupo5

Revert "update"

d2bc890

This reverts commit f80c828.

alvaroaxsmith and others added 5 commits May 5, 2022 12:00

Merge pull request #7 from alvaroaxsmith/dependabot/npm_and_yarn/mini…

13d0066

…mist-1.2.6 Bump minimist from 1.2.5 to 1.2.6

Merge pull request #9 from alvaroaxsmith/dependabot/npm_and_yarn/asyn…

01e5744

…c-2.6.4 Bump async from 2.6.3 to 2.6.4

Merge pull request #8 from alvaroaxsmith/dependabot/npm_and_yarn/even…

ca902e7

…tsource-1.1.1 Bump eventsource from 1.1.0 to 1.1.1

Conversation

giovaner10 commented Mar 27, 2022 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

renato3x left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

renato3x commented Mar 28, 2022

Uh oh!

giovaner10 commented Mar 28, 2022

Uh oh!

tikaoo commented Apr 4, 2022

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

renato3x commented Apr 5, 2022

Uh oh!

renato3x commented Apr 11, 2022

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

8 participants

Comments

giovaner10 commented Mar 27, 2022 •

edited

Loading