Skip to content

Comments

[ST-F-010] Создал страницу регистрации пользователя#18

Open
VovaDVA wants to merge 3 commits intomasterfrom
ST-F-010
Open

[ST-F-010] Создал страницу регистрации пользователя#18
VovaDVA wants to merge 3 commits intomasterfrom
ST-F-010

Conversation

@VovaDVA
Copy link
Collaborator

@VovaDVA VovaDVA commented Dec 9, 2023

No description provided.

@Aleksio02 Aleksio02 changed the title Создал страницу регистрации пользователя [ST-F-010] Создал страницу регистрации пользователя Dec 9, 2023
Comment on lines 1 to 23
.DS_Store
node_modules
/dist


# local env files
.env.local
.env.*.local

# Log files
npm-debug.log*
yarn-debug.log*
yarn-error.log*
pnpm-debug.log*

# Editor directories and files
.idea
.vscode
*.suo
*.ntvs*
*.njsproj
*.sln
*.sw?
Copy link
Owner

Choose a reason for hiding this comment

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

в проекте должен быть только 1 .gitignore (в корне проекта)

Comment on lines -7 to -9
"dev": "vite",
"build": "vite build",
"preview": "vite preview"
Copy link
Owner

Choose a reason for hiding this comment

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

почему ты удалил эти скрипты?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Я ничего не удалял, просто создал новое приложение через vue-cli. У тебя оно создано на Vite, а у меня на Cli, поэтому нет таких строк.

Copy link
Owner

Choose a reason for hiding this comment

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

Раздели этот файл на компоненты vue. Компонент Vue можно использовать несколько раз на разных страницах.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Он и так вроде разделен, для каждой страницы свой компонент, общая верстка только в App.js. Для регистрации, авторизации и других страниц свои компоненты

Copy link
Owner

@Aleksio02 Aleksio02 Dec 10, 2023

Choose a reason for hiding this comment

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

Это надо сделать в рамках другой таски

Copy link
Owner

Choose a reason for hiding this comment

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

Раздели файл на компоненты Vue (например, вынеси поле логина и пароля в отдельный компонент). Компонент Vue можно использовать несколько раз на разных страницах

Copy link
Owner

Choose a reason for hiding this comment

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

укажи здесь порт 9000

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Поставил порт

Copy link
Owner

Choose a reason for hiding this comment

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

эта форма не в этой таске делается

Copy link
Owner

Choose a reason for hiding this comment

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

текстовые поля можно вынести в отдельные компоненты (например: PasswordField, LoginField)

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Не вижу в этом смысла, по идее форма будет только на спецстраницах

Copy link
Owner

Choose a reason for hiding this comment

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

если этот компонент занимает всю страницу, то его надо называть RegistrationPage и вынести в пакет src/pages (т.е.рядом с пакетом компонентов)

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