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.

Sign up for GitHub

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

Jump to bottom

Consolidate /badges and /credentialsroutes #284

Closed
maiertech opened this issue Jul 15, 2024 · 0 comments · Fixed by #366
Closed

Consolidate /badges and /credentialsroutes #284

maiertech opened this issue Jul 15, 2024 · 0 comments · Fixed by #366
Assignees
@maiertech
Labels
Frontend This issue pertains to the UniMe frontend. Refactor Refactor code without changing functionality.
Milestone
UniMe Quick Wins

Comments

@maiertech
Copy link
Member

Description

Both routes essentially render the same thing. /badges is the newer design. The consolidated route should be called /credentials.

Motivation

No response

Resources

The refactored route should also fix #233.

To-do List

n/a
@maiertech maiertech added Frontend This issue pertains to the UniMe frontend. Refactor Refactor code without changing functionality. labels Jul 15, 2024
@maiertech maiertech self-assigned this Jul 15, 2024
@maiertech maiertech mentioned this issue Jul 15, 2024
Closed
6 tasks
@JelleMillenaar JelleMillenaar added this to the UniMe Quick Wins milestone Jul 29, 2024
@daniel-mader daniel-mader mentioned this issue Sep 3, 2024
Merged
6 tasks
@maiertech maiertech mentioned this issue Sep 6, 2024
Merged
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@maiertech maiertech

Labels
Frontend This issue pertains to the UniMe frontend. Refactor Refactor code without changing functionality.
Projects
Impierce SSI Stack
Status: Done
Milestone
UniMe Quick Wins
Development

Successfully merging a pull request may close this issue.

refactor: Consolidate /badges and /credentials routes impierce/identity-wallet
2 participants
@maiertech @JelleMillenaar