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

III-5851 - Implementation new design #815

Merged
merged 54 commits into from
Nov 14, 2023
Merged

III-5851 - Implementation new design #815

merged 54 commits into from
Nov 14, 2023

Conversation

brampauwelyn
Copy link
Contributor

@brampauwelyn brampauwelyn commented Oct 18, 2023

Changed

  • Implementation new design

Screenshots:
Screenshot 2023-11-07 at 14 51 08

Screenshot 2023-11-06 at 12 54 09 Screenshot 2023-11-07 at 13 56 12

Angular part: cultuurnet/udb3-angular-app#205


Ticket: https://jira.uitdatabank.be/browse/III-5851

@brampauwelyn brampauwelyn marked this pull request as ready for review November 8, 2023 10:38
@brampauwelyn brampauwelyn requested a review from vhande November 8, 2023 11:27
src/layouts/Sidebar.tsx Outdated Show resolved Hide resolved
src/layouts/Sidebar.tsx Outdated Show resolved Hide resolved
src/pages/steps/Steps.tsx Outdated Show resolved Hide resolved
const { asPath } = useRouter();

const baseUrl = publicRuntimeConfig.baseUrl;
const currentPath = new URL(asPath, baseUrl).pathname;
Copy link
Contributor

Choose a reason for hiding this comment

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

Doesn't useRouter also expose a normal path?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I don't remember exactly why but I wasn't able to do what I wanted with the variables exposed by useRouter.

@brampauwelyn brampauwelyn merged commit 9f676dd into main Nov 14, 2023
5 checks passed
@brampauwelyn brampauwelyn deleted the feature/III-5851 branch November 14, 2023 09:07
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.

3 participants