Skip to content

Cs 10291 create submission card portal#4103

Open
lucaslyl wants to merge 28 commits intomainfrom
CS-10291-create-submission-card-portal
Open

Cs 10291 create submission card portal#4103
lucaslyl wants to merge 28 commits intomainfrom
CS-10291-create-submission-card-portal

Conversation

@lucaslyl
Copy link
Contributor

@lucaslyl lucaslyl commented Mar 3, 2026

linear: https://linear.app/cardstack/issue/CS-10291/create-submission-card-portal

Screen.Recording.2026-03-12.at.8.19.29.PM.mov

@chatgpt-codex-connector
Copy link

To use Codex here, create a Codex account and connect to github.

@lucaslyl
Copy link
Contributor Author

lucaslyl commented Mar 3, 2026

@codex review

@lucaslyl lucaslyl marked this pull request as ready for review March 3, 2026 06:11
@lucaslyl lucaslyl marked this pull request as draft March 3, 2026 06:13
Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 779a18d413

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@github-actions
Copy link

github-actions bot commented Mar 3, 2026

Host Test Results

0 files   -     1  0 suites   - 1   0s ⏱️ - 2h 23m 1s
0 tests  - 2 010  0 ✅  - 1 995  0 💤  - 15  0 ❌ ±0 
0 runs   - 2 025  0 ✅  - 2 010  0 💤  - 15  0 ❌ ±0 

Results for commit 8988c9e. ± Comparison against base commit 3665973.

♻️ This comment has been updated with latest results.

@lucaslyl
Copy link
Contributor Author

@codex review

@lucaslyl lucaslyl requested a review from a team March 10, 2026 15:10
@lucaslyl lucaslyl self-assigned this Mar 10, 2026
Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 4e9ba01196

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment on lines +243 to +244
get prCardInstance() {
return this.prCardData?.instances?.[0] ?? null;

Choose a reason for hiding this comment

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

P2 Badge Sort PR-card search before taking first result

prCardInstance always selects instances[0], but the query only filters by branchName and does not define a sort order. The index query engine defaults unsorted searches to ORDER BY url (packages/runtime-common/index-query-engine.ts:583-586), so if multiple PR cards exist for the same branch (for example after closing and reopening a PR), this can resolve to an older card instead of the current one; please add an explicit descending sort (e.g., by lastModified or submittedAt) before reading the first instance.

Useful? React with 👍 / 👎.

@lucaslyl lucaslyl marked this pull request as ready for review March 11, 2026 03:17
@lucaslyl lucaslyl force-pushed the CS-10291-create-submission-card-portal branch from 607ff12 to afa3185 Compare March 11, 2026 13:22
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.

1 participant