Skip to content

List decisions on the steward's profile page#602

Merged
scazan merged 8 commits intodevfrom
list-decision-on-steward
Feb 14, 2026
Merged

List decisions on the steward's profile page#602
scazan merged 8 commits intodevfrom
list-decision-on-steward

Conversation

@scazan
Copy link
Collaborator

@scazan scazan commented Feb 13, 2026

Updates our decisions tab.

  • Removes dependency on "modules" and shows if any (that user has access to) exist
  • Shows decision processes in which a profile is a steward (org or individual)
  • Adds green dot for active process notification
  • Handles legacy processes with intention of removal so having the endpoint fail gracefully on validation errors until then
Screenshot 2026-02-13 at 10 39 31

@vercel
Copy link
Contributor

vercel bot commented Feb 13, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
app Ready Ready Preview Feb 14, 2026 5:42pm

Request Review


const hasDecisionProfiles = (decisionProfiles.data?.items?.length ?? 0) > 0;
const hasLegacyInstances = (legacyInstances.data?.instances?.length ?? 0) > 0;
const hasPublishedDecisions = decisionProfiles.data?.items?.some(
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Might need to restructure this a bit on the backend since ordering needs to push published to the top. Followup PR for that.

);

if (!hasDecisionProfiles && !hasLegacyInstances) {
return null;
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Don't show the tab

total: result.total,
hasMore: result.hasMore,
}))
.filter((instance) => {
Copy link
Collaborator Author

@scazan scazan Feb 13, 2026

Choose a reason for hiding this comment

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

We filter ones that don't pass because we are unfortunately sharing space with new and old. This is only the case for three known processes that will be migrated. This prevents any breakage as a result.

@scazan scazan marked this pull request as ready for review February 13, 2026 10:35
@scazan scazan force-pushed the list-decision-on-steward branch from a762360 to 1261164 Compare February 14, 2026 17:40
@scazan scazan merged commit 8f11db4 into dev Feb 14, 2026
10 checks passed
@scazan scazan deleted the list-decision-on-steward branch February 14, 2026 18:50
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.

2 participants