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

Add achievement list page #304

Merged
merged 5 commits into from Sep 14, 2023
Merged

Conversation

danokp
Copy link
Contributor

@danokp danokp commented Aug 6, 2023

Resolves #265

@fey
Copy link
Collaborator

fey commented Aug 7, 2023

@danokp Добрый день.

У вас появился конфликт из-за мержа другого ПР (его смержили в оригинальный репозиторий). Попробуйте разрешить конфликт самостоятельно.

@fey
Copy link
Collaborator

fey commented Aug 14, 2023

@danokp Добрый день. Получается разобраться с конфликтом?

@danokp
Copy link
Contributor Author

danokp commented Aug 14, 2023

@fey , добрый день!
Ага, вроде получилось. Я забыл нажать commit merge :(

@fey
Copy link
Collaborator

fey commented Aug 14, 2023

@danokp падает пайплайн, посморите пожалуйтса.

@danokp
Copy link
Contributor Author

danokp commented Aug 14, 2023

@fey буду разбираться:)

@dzencot
Copy link

dzencot commented Aug 28, 2023

Привет! Получается? Нужна помощь?

@fey fey marked this pull request as draft September 5, 2023 13:12
@danokp danokp marked this pull request as ready for review September 5, 2023 19:26
@danokp
Copy link
Contributor Author

danokp commented Sep 5, 2023

@fey @dzencot Привет!
Прошу прощения, что пропал. Внес изменения: сейчас пайплайн не падает.

@fey
Copy link
Collaborator

fey commented Sep 6, 2023

@danokp задеплойте, пожалуйста на render

@ola-9
Copy link

ola-9 commented Sep 11, 2023

@danokp как у вас успехи с деплоем, нужна какая-нибудь помощь?

@DREU007
Copy link
Contributor

DREU007 commented Sep 13, 2023

@fey, @ola-9 Задеплоил https://hexlet-friends-issue265.onrender.com

@fey
Copy link
Collaborator

fey commented Sep 14, 2023

ХммМ, а как попасть на страницу ачивок? 🤔
Кажется нашел, через страницу профиля.

@fey
Copy link
Collaborator

fey commented Sep 14, 2023

@DREU007 я этот ПР приму. Можете написать тесты на новую страницу с ачивками?

@fey fey merged commit 604f980 into Hexlet:main Sep 14, 2023
1 check passed
@fey
Copy link
Collaborator

fey commented Sep 26, 2023

@danokp Привет, можете добавить тесты на путь который добавили? Хорошо, когда мы не просто что-то добавляем, но и еще пишем тесты на это.

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.

Add achievements page
5 participants