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

feat: [AXIMST-38] Unit page - update breadcrumbs #92

Conversation

ihor-romaniuk
Copy link
Collaborator

@ihor-romaniuk ihor-romaniuk commented Jan 5, 2024

Description: update breadcrumbs on the Course Unit page

Youtrack: https://youtrack.raccoongang.com/issue/AXIMST-38

@ihor-romaniuk ihor-romaniuk force-pushed the romaniuk/feat/AXIMST-38/update-breadcrumbs-on-course-unit-page branch from 97c2537 to 8bb2745 Compare January 5, 2024 10:09
@ihor-romaniuk ihor-romaniuk changed the base branch from master to ts-develop January 5, 2024 10:28
@ihor-romaniuk ihor-romaniuk force-pushed the romaniuk/feat/AXIMST-38/update-breadcrumbs-on-course-unit-page branch from 8bb2745 to be32da7 Compare January 5, 2024 12:34
src/course-outline/hooks.jsx Outdated Show resolved Hide resolved
src/course-unit/breadcrumbs/Breadcrumbs.jsx Show resolved Hide resolved
src/course-unit/breadcrumbs/Breadcrumbs.test.jsx Outdated Show resolved Hide resolved
@monteri monteri force-pushed the ts-develop branch 2 times, most recently from 3ec4b3f to 387b188 Compare January 5, 2024 17:14
@GlugovGrGlib
Copy link
Collaborator

GlugovGrGlib commented Jan 6, 2024

Please rebase your commits on top of ts-develop and then force push it, so there will be only feat: update breadcrumbs on course unit page in PR

Additional Code Review fixes, please add in separate commit (test: or fix:)

@ihor-romaniuk ihor-romaniuk force-pushed the romaniuk/feat/AXIMST-38/update-breadcrumbs-on-course-unit-page branch 2 times, most recently from 596e26a to 200e908 Compare January 8, 2024 14:42
@ihor-romaniuk ihor-romaniuk force-pushed the romaniuk/feat/AXIMST-38/update-breadcrumbs-on-course-unit-page branch from 200e908 to adef432 Compare January 8, 2024 14:53
@monteri monteri merged commit 10e4dc0 into ts-develop Jan 8, 2024
2 of 3 checks passed
@ihor-romaniuk ihor-romaniuk changed the title feat: Unit page - update breadcrumbs feat: [AXIMST-38] Unit page - update breadcrumbs Jan 8, 2024
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.

4 participants