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
/event-types/detail wrong empty screen for workflows #4757
Conversation
@gitstart is attempting to deploy a commit to the cal Team on Vercel. A member of the Team first needs to authorize it. |
Hi @PeerRich! Kindly review the PR |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for your contribution 🙏 I have noticed two things that need some changes:
- It should be aligned with the settings tabs (compare with empty screen of apps, should look the same)
- The empty screen of /workflows also got changed, it should stay as is it is right now. So we want it to look like that:
Thank you @CarinaWolli for the review, will make the required changes and get back to you |
Co-authored-by: gitstart <gitstart@gitstart.com> Co-authored-by: gitstart <gitstart@users.noreply.github.com> Co-authored-by: Rafael Toledo <87545086+Toledodev@users.noreply.github.com> Co-authored-by: Matheus Muniz <matheusmuniz100@hotmail.com> Co-authored-by: Matheus Muniz <87545749+matheusmuniz03@users.noreply.github.com> Co-authored-by: Thiago Nascimbeni <tnascimbeni@gmail.com> Co-authored-by: Klinger Matheus <50892465+KlingerMatheus@users.noreply.github.com> Co-authored-by: Nitesh Singh <nitesh.singh@gitstart.dev> Co-authored-by: Grace Nshokano <grace.devolop@gmail.com> Co-authored-by: Olusanya Timothy <48022904+seunexplicit@users.noreply.github.com>
Hi @CarinaWolli! PR was updated with the changes, kindly review again |
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great job, looks good now!
What does this PR do?
Fixes #4265
Demo
Loom:
https://www.loom.com/share/39b25ddceca045c2a98251bbf8f05cb2
Old Version:
Updated Version:
Environment: Staging(main branch) / Production
Type of change