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

Estruturação do Projeto para ser uma SPA #256

Merged
merged 10 commits into from
Jun 11, 2024
Merged

Conversation

muritadb
Copy link
Contributor

Closes #251

Feature

Estruturação do Projeto para ser uma SPA

Visual evidences 🖼️

image

Checklist
  • Issue linked
  • Build working correctly
  • Tests created
Additional info Alterei a Home para Pages e dentro Components para layouts.

@muritadb muritadb force-pushed the issue-251 branch 2 times, most recently from 052e8db to dc71462 Compare May 30, 2024 19:43
@muritadb muritadb changed the title Issue 251 Estruturação do Projeto para ser uma SPA May 30, 2024
Copy link
Contributor

@Alecell Alecell left a comment

Choose a reason for hiding this comment

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

Ficou ao contrário acho, não deveria ser um layout que possui pages? Um layout seria um formato de página, um formato de página tem várias paginas.

No fim podemos ter vários layouts diferentes, cada um com várias possibilidades de páginas.

Pensa que o layout é uma estrutura da página em si e a página e o conteúdo que será exibido na estrutura

src/pages/index.html Outdated Show resolved Hide resolved
src/pages/index.js Outdated Show resolved Hide resolved
src/pages/utils/scrollable-sidemenu.js Outdated Show resolved Hide resolved
Copy link
Contributor

@Alecell Alecell left a comment

Choose a reason for hiding this comment

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

Dentro de cada layout vamos ter uma pasta chamada components, esses components definem componentes especificos do layout, dentro de layout tambem há uma pasta chamada pages

SideMenu e Navigation são componentes do layout PetDexApp
NoPetRegisteredPage é uma página

Valendo dizer, vamos padronizar essa nomenclatura. A partir de agora tudo que for página não terá mais o suffixo Page, saberemos oque é página ou não decorrente de estar dentro da pasta pages!

image

src/pages/images/pet-dex.svg Outdated Show resolved Hide resolved
@muritadb
Copy link
Contributor Author

image

Bom a estrutura ficou dessa forma:
Layouts - para cada layout do projeto
PetDexApp - layout existente
PetDexApp -> Components - componentes utilizados nesse layout
PetDexApp -> pages - variações dessa pagina
Utils - componentes uteis do layout

@Alecell Alecell merged commit 82183bf into devhatt:main Jun 11, 2024
3 checks passed
JonasGz added a commit to JonasGz/pet-dex-frontend-p4n4 that referenced this pull request Jun 15, 2024
* finish service routes

remove codes

* chore: changed pt-br to english

* style: add eof package.json

* feat: change home folder to pages

* feat: change page petRegisterPage story  structure imports

* fix: change index directory to pages/

* fix: resolve conflict changes sidemenu

* refactor: change directories name of SPA structure

* refactor: refactor in pages and components for nomenclature correct

* refactor: moving Utils for PetDexApp directory

---------

Co-authored-by: JonasGz <[email protected]>
JonasGz added a commit to JonasGz/pet-dex-frontend-p4n4 that referenced this pull request Jun 15, 2024
* finish service routes

remove codes

* chore: changed pt-br to english

* style: add eof package.json

* feat: change home folder to pages

* feat: change page petRegisterPage story  structure imports

* fix: change index directory to pages/

* fix: resolve conflict changes sidemenu

* refactor: change directories name of SPA structure

* refactor: refactor in pages and components for nomenclature correct

* refactor: moving Utils for PetDexApp directory

---------

Co-authored-by: JonasGz <[email protected]>
JonasGz added a commit to JonasGz/pet-dex-frontend-p4n4 that referenced this pull request Jun 15, 2024
* finish service routes

remove codes

* chore: changed pt-br to english

* style: add eof package.json

* feat: change home folder to pages

* feat: change page petRegisterPage story  structure imports

* fix: change index directory to pages/

* fix: resolve conflict changes sidemenu

* refactor: change directories name of SPA structure

* refactor: refactor in pages and components for nomenclature correct

* refactor: moving Utils for PetDexApp directory

---------

Co-authored-by: JonasGz <[email protected]>
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.

Estruturar o projeto para ter pastas como uma SPA
4 participants