Skip to content

@W-21650070 fix: unhide webapps from sf --help#878

Merged
iowillhoit merged 2 commits intomainfrom
lt/unhide-webapp-and-fix-project-types
Mar 30, 2026
Merged

@W-21650070 fix: unhide webapps from sf --help#878
iowillhoit merged 2 commits intomainfrom
lt/unhide-webapp-and-fix-project-types

Conversation

@Shinoni
Copy link
Copy Markdown
Collaborator

@Shinoni Shinoni commented Mar 20, 2026

What does this PR do?

Remove the hidden flag for webapps templates from sf --help

@W-21650070@

What issues does this PR fix or reference?

@Shinoni Shinoni requested a review from mshanemc March 20, 2026 23:06
@Shinoni Shinoni requested a review from bpbuch March 20, 2026 23:20
manifest: flags.manifest,
loginurl: flags['login-url'],
template: flags.template,
template: flags.template as ProjectOptions['template'],
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Why is it needed here? it works as expected, I tested it and I don't see any type mismatch.

@iowillhoit iowillhoit marked this pull request as draft March 23, 2026 16:30
@Shinoni Shinoni marked this pull request as ready for review March 30, 2026 14:27
@garychangsf garychangsf force-pushed the lt/unhide-webapp-and-fix-project-types branch from 8317543 to f6b2472 Compare March 30, 2026 18:56
@iowillhoit
Copy link
Copy Markdown
Contributor

Dependabot warning is unrelated. I have pinged @k-j-kim about it.
Merging this PR

@iowillhoit iowillhoit merged commit 80dfa80 into main Mar 30, 2026
15 of 16 checks passed
@iowillhoit iowillhoit deleted the lt/unhide-webapp-and-fix-project-types branch March 30, 2026 19:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants