Skip to content

feat(aci): add type and creator cell components #88862

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

Merged
merged 4 commits into from
Apr 10, 2025
Merged

Conversation

natemoo-re
Copy link
Member

@natemoo-re natemoo-re commented Apr 4, 2025

Adds two new cells to the Workflow Engine components—type and user cell

Screenshot 2025-04-07 at 5 43 59 PM

@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Apr 4, 2025
@ameliahsu
Copy link
Member

are we adding the columns to the index page in this PR? or just the gridCells?

@natemoo-re
Copy link
Member Author

@ameliahsu just adding the grid cells in this PR, since my other branch has the types refactor and also cleans up table props.

@natemoo-re natemoo-re changed the title feat(aci): add type and creator columns feat(aci): add type and creator cell components Apr 7, 2025
@natemoo-re natemoo-re marked this pull request as ready for review April 7, 2025 22:44
@natemoo-re natemoo-re requested a review from a team as a code owner April 7, 2025 22:44
@natemoo-re natemoo-re requested a review from ameliahsu April 7, 2025 22:44
disabled?: boolean;
};

export function UserCell({user, disabled = false, className}: UserCellProps) {
Copy link
Member

@ameliahsu ameliahsu Apr 7, 2025

Choose a reason for hiding this comment

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

should there be a tooltip here with the user's email or name? not sure if this is the expected behavior when you hover 😅

Screenshot 2025-04-07 at 4 02 53 PM

Copy link

codecov bot commented Apr 9, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

✅ All tests successful. No failed tests found.

Additional details and impacted files
@@             Coverage Diff             @@
##           master   #88862       +/-   ##
===========================================
- Coverage   86.49%   75.25%   -11.25%     
===========================================
  Files       10107    10107               
  Lines      571949   571949               
  Branches    22474    22474               
===========================================
- Hits       494723   430403    -64320     
- Misses      76789   141109    +64320     
  Partials      437      437               

@natemoo-re natemoo-re enabled auto-merge (squash) April 10, 2025 00:06
@natemoo-re natemoo-re merged commit 66d0e00 into master Apr 10, 2025
41 checks passed
@natemoo-re natemoo-re deleted the aci/nm/columns branch April 10, 2025 00:12
MichaelSun48 pushed a commit that referenced this pull request Apr 11, 2025
Adds two new cells to the Workflow Engine components—type and user cell

<img width="357" alt="Screenshot 2025-04-07 at 5 43 59 PM"
src="https://github.com/user-attachments/assets/bd21cdef-a4af-4541-92a8-9b1fbb135cf2"
/>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Scope: Frontend Automatically applied to PRs that change frontend components
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants