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

portalicious: add registrations breadcrumb and handle not found #5882

Merged
merged 1 commit into from
Oct 3, 2024

Conversation

aberonni
Copy link
Contributor

@aberonni aberonni commented Oct 1, 2024

AB#30165
AB#30395

I combined these two tasks because I found one relying on the other, and figured that they wouldn't be a large set of changes even when combined.

The design of the "registration not found" deviates from what is in Figma based on a convo with Tal.

image

I also took the liberty of adding a "project not found" error.

Checklist before requesting a review

  • I have performed a self-review of my code
  • I have added tests wherever relevant
  • I have made sure that all automated checks pass before requesting a review
  • I do not need any deviation from our PR guidelines

@aberonni aberonni added the portalicious Changes related to the portalicious release label Oct 1, 2024
@aberonni aberonni marked this pull request as ready for review October 1, 2024 12:47
@aberonni aberonni force-pushed the aberonni/portalicious-breadcrumbs-and-not-found branch from cc502d3 to 8cdc413 Compare October 1, 2024 14:14
@aberonni aberonni force-pushed the aberonni/portalicious-breadcrumbs-and-not-found branch from 8cdc413 to e7ac5b4 Compare October 1, 2024 15:23
@aberonni aberonni requested a review from arsforza October 2, 2024 08:15
@aberonni aberonni force-pushed the aberonni/portalicious-breadcrumbs-and-not-found branch from e7ac5b4 to 59770f7 Compare October 2, 2024 08:33
@aberonni aberonni enabled auto-merge (squash) October 2, 2024 08:33
@aberonni aberonni force-pushed the aberonni/portalicious-breadcrumbs-and-not-found branch from 59770f7 to 5c9932f Compare October 2, 2024 11:46
@aberonni aberonni force-pushed the aberonni/portalicious-breadcrumbs-and-not-found branch from 5c9932f to 029732e Compare October 2, 2024 13:37
@aberonni aberonni force-pushed the aberonni/portalicious-breadcrumbs-and-not-found branch from 029732e to 0349ba2 Compare October 2, 2024 14:11
AB#30165
AB#30395

I combined these two tasks because I found one relying on the other, and figured that they wouldn't be a large set of changes even when combined.

The design of the "registration not found" deviates from what is in Figma based on a convo with Tal.

I also took the liberty of adding a "project not found" error.
@aberonni aberonni merged commit f439a9b into main Oct 3, 2024
5 checks passed
@aberonni aberonni deleted the aberonni/portalicious-breadcrumbs-and-not-found branch October 3, 2024 09:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
portalicious Changes related to the portalicious release
Development

Successfully merging this pull request may close these issues.

2 participants