feat(site): add ModuleSelectStep - #26427
Merged
Merged
Conversation
This was referenced Jun 16, 2026
Contributor
Author
This was referenced Jun 16, 2026
jeremyruppel
commented
Jun 16, 2026
| // TODO add this to the API response so we don't have to construct it manually here. | ||
| function moduleDetailsUrl(moduleId: string): string { | ||
| return `https://registry.coder.com/modules/${moduleId}`; | ||
| } |
Contributor
Author
There was a problem hiding this comment.
comment is correct. i'll add this to the backend in a future PR
jeremyruppel
marked this pull request as ready for review
June 16, 2026 17:53
jeremyruppel
changed the title
jeremyruppel
requested review from
aqandrew,
aslilac,
code-asher and
zenithwolf1000
June 16, 2026 18:02
This was referenced Jun 16, 2026
jeremyruppel
force-pushed
the
jeremy/devex-284-fe-basetemplateparametersstep
branch
from
June 16, 2026 20:42
27cd694 to
94f5fb5
Compare
jeremyruppel
force-pushed
the
jeremy/devex-285-fe-moduleselectstep
branch
from
June 16, 2026 20:42
32f0468 to
ec336be
Compare
jeremyruppel
force-pushed
the
jeremy/devex-284-fe-basetemplateparametersstep
branch
from
June 17, 2026 15:59
94f5fb5 to
25ac414
Compare
jeremyruppel
force-pushed
the
jeremy/devex-285-fe-moduleselectstep
branch
from
June 17, 2026 15:59
ec336be to
612d943
Compare
Base automatically changed from
jeremy/devex-284-fe-basetemplateparametersstep
to
main
June 17, 2026 18:56
Implement the module selection wizard step: - Add getTemplateBuilderModules API client method with optional base parameter for OS filtering - Add react-query wrapper with staleTime: Infinity to prevent re-fetches on step navigation - Add ModuleSelectStep component that renders a flat grid of ModuleCard components with multi-select toggle - Show non-blocking conflict warnings when selected modules declare conflicts with each other - Wire the step into TemplateBuilderPageView for the module-select step ID - Export SelectedModuleMeta from wizardState.ts for the step callback
jeremyruppel
force-pushed
the
jeremy/devex-285-fe-moduleselectstep
branch
from
June 17, 2026 18:58
612d943 to
0094680
Compare
zenithwolf1000
approved these changes
Jun 17, 2026
aqandrew
approved these changes
Jun 18, 2026
aqandrew
left a comment
aqandrew
left a comment
Contributor
There was a problem hiding this comment.
2 small nits, otherwise lgtm 🙂
…ools Co-authored-by: Andrew Aquino <dawneraq@gmail.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Implement the module selection wizard step with multi-select toggle and conflict warnings.
getTemplateBuilderModulesAPI client method with optionalbaseparameter for OS filteringstaleTime: Infinityto prevent re-fetches on step navigationModuleCardcomponents with checkbox-style multi-selectconflicts_witheach otherTemplateBuilderComposeModule[]andSelectedModuleMeta[]for wizard stateRelates to DEVEX-285.
Note
This PR was authored with Coder Agents.