fix: improve demo experience - #1203
Conversation
|
Install the latest version of pnpm add https://pkg.svelte.dev/sv/c/adf26ee7d6f923c34fb5412cddf2498dab3c9de7Open in Note This PR is from a fork. A maintainer must approve approve each commit before it can be built and installed. |
|
| ], | ||
| required: false | ||
| required: false, | ||
| condition: ({ template }) => template !== 'demo' |
There was a problem hiding this comment.
@jycouet the general idea of this PR is this, thoughts?
There was a problem hiding this comment.
- I find it strange to not allow better-auth demo in demo template
There was a problem hiding this comment.
The logic got lost during the rebasing. It's supposed to be "when template is demo, don't ask, always use template"
There was a problem hiding this comment.
The logic got lost during the rebasing. It's supposed to be "when template is demo, don't ask, always use template"
I guess you meant when template is demo, don't ask, always use demo ?
I don't really see the value there and it's doing a big breaking, and it's only for create.
jycouet
left a comment
There was a problem hiding this comment.
All in all, I'm not a big fan.
I re read the issue a few times, and:
On the demo template, add a fourth header called addon which links to the /demo page.
It could also be /demo as a 4th tab?
Actually, in the minimal template, I would love to have a link to the demo page from the root layout! (It's maybe VERY intrusive... but very visible for people adding something in their app!)
when sv create, addons should not prompt to add demo if the template is demo
I think that this is not necessary. Why doing "soooo much work" for so small outcome? Maybe I'm missing somehting?
rename /demo to /addon
Possible yes, but I don't really see the cool thing about it
621319f to
6624f6a
Compare
6624f6a to
e9d82d0
Compare
f17c98a to
e1892a4
Compare
|
changes:
Overall, the cumulative diff has been reduced so its easier to see the intend. I think less prompting is better. Especially when it comes to demos, which is used by new users. And it's a redundant question. api changes
|
Closes #1169
Description
Reopened from #1188 due to a glitch.
Checklist