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

[#381] added link to remindPassword to SignInForm.jsx #383

Closed
wants to merge 5 commits into from
Closed

[#381] added link to remindPassword to SignInForm.jsx #383

wants to merge 5 commits into from

Conversation

elenashcherbinina
Copy link
Contributor

добавила "не помню пароль" на страницу авторизации.

было:
было

стало:
стало

@fey
Copy link
Contributor

fey commented Sep 26, 2023

@elenashcherbinina у вас конфликт в ПРе, получится самостоятельно пофиксить?

@fey fey requested a review from dzencot September 26, 2023 11:11
@elenashcherbinina
Copy link
Contributor Author

пофиксила конфликты, осталось только мое изменение в файле frontend/src/components/Forms/SignInForm.jsx

@fey
Copy link
Contributor

fey commented Sep 26, 2023

@elenashcherbinina кажется конфликт все равно не пофикшен 🤔

@elenashcherbinina
Copy link
Contributor Author

Может я тогда не понимаю что-то))

Я внесла правки локально в документ frontend/src/components/Forms/SignInForm.jsx. Потом запушила в новую ветку и сделала пулреквест. Так как кто-то после этого работал в файлах, были конфликты, я их исправила. Сейчас конфликтом являются мои правки. Мы же хотим, чтобы 'не помню пароль' было видно сразу. Но если я сейчас этот конфликт исправлю, то я получается снова вернусь к первоначальной версии)
То есть на данный момент я пока не понимаю, как вносить изменения, чтобы не было кинфликта. Ведь любое изменение - это уже конфликт)
Коля, извини за вопросы, я пока только учусь )

@fey
Copy link
Contributor

fey commented Sep 27, 2023

@elenashcherbinina смотрите, конфликты происходят из-за того, что у нас появляются разные коммиты с изменениями в одном и том же месте. Иногда гит может разрулить изменения, иногда нет.

Обычно я разруливаю изменения командой git pull --rebase upstram main, скачиваю в свою ветку изменения, мерж коммита нет. Происходит ребейз, там где происходит конфликт, начинается его разруливание, и нужно выбрать те изменения, которые нужны. Это продвинутая техника, более простая - сделать git pull - будет скорее всего создан мерж коммит и изменения подтянутся, нужно будет также порешать конфликт.

@fey
Copy link
Contributor

fey commented Sep 27, 2023

Давайте попросим Лену помочь вам) @HelenOne помоги пожалуйста Лене разобраться с конфликтом. Возможно тут хороший повод для масскодинга намечается.

@fey fey requested a review from HelenOne September 27, 2023 07:40
@fey fey marked this pull request as draft September 27, 2023 07:40
@elenashcherbinina elenashcherbinina marked this pull request as ready for review September 28, 2023 06:58
@elenashcherbinina
Copy link
Contributor Author

Всем привет! Я залила все изменения, которые произошли в мой репозиторий. Я сначала все конфликты решала самостоятельно, поэтому сейчас у меня в пулреквесте 9 измененных файлов, хотя я сама изменяла только один: frontend/src/components/Forms/SignInForm.jsx.

Можно мне как-то удалить этот пулреквест и попробовать все заново?

@HelenOne
Copy link
Contributor

Привет! Удалить нельзя, но можем закрыть и попробуешь заново
Уверена?

@fey
Copy link
Contributor

fey commented Sep 28, 2023

Попробуйте сделать ребейз и удалить коммиты с резолвом, затем сделать форспуш.

@elenashcherbinina elenashcherbinina closed this by deleting the head repository Sep 28, 2023
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.

None yet

3 participants