Add issue tracker cards and project-backed kanban board to software-factory#4650
Add issue tracker cards and project-backed kanban board to software-factory#4650burieberry wants to merge 23 commits intomainfrom
Conversation
Host Test Results 1 files ±0 1 suites ±0 1h 42m 29s ⏱️ - 3m 24s Results for commit 9f09e9e. ± Comparison against earlier commit d0d9fd9. Realm Server Test Results 1 files ±0 1 suites ±0 15m 27s ⏱️ - 2m 7s Results for commit 9f09e9e. ± Comparison against earlier commit d0d9fd9. |
b08a0e2 to
942b3c1
Compare
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: d49d7990eb
ℹ️ 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".
lukemelia
left a comment
There was a problem hiding this comment.
Do we want test coverage for this?
move docs set experiments-realm issue tracker as symlink
b80c6d1 to
d5209e5
Compare
Updated software-factory operations and tests to include board creation. Component itself has tests in boxel-ui. More card tests can be added when cards are moved out of software-factory. |
habdelra
left a comment
There was a problem hiding this comment.
Probably a better test would be to use a playwright spec test in the software factory to test the actual rendering of the new card defs, test moving the issues between column sin the kanban board, adding comments asserting the comments appear in the issue, etc. like full integration tests via the DOM. the node tests are mostly cheating in this regard because they would rely on mocks of the realm server and don't actually try to render the card.
d0d9fd9 to
9f09e9e
Compare
Yes, I want to add ui tests after moving it out of software-factory because it's been a huge struggle to be able to write them as playwright tests. Other option is I move them out now. The kanban component itself has tests in boxel-ui. |
sounds good--i trust your judgement |
Summary
Next Steps: