Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[🐞] qwik new CamelCaseComponent exported component becomes Camelcasecomponent #5282

Closed
chug2k opened this issue Oct 7, 2023 · 2 comments · Fixed by #5430
Closed

[🐞] qwik new CamelCaseComponent exported component becomes Camelcasecomponent #5282

chug2k opened this issue Oct 7, 2023 · 2 comments · Fixed by #5430
Labels
COMMUNITY: PR is welcomed We think it's a good feature to have but would love for the community to help with the PR for it COMP: starters TYPE: bug Something isn't working

Comments

@chug2k
Copy link

chug2k commented Oct 7, 2023

Which component is affected?

Starters / CLI

Describe the bug

I created a new component using qwik new and I lost camelcase in the Props and export declaration.

Reproduction

n/a

Steps to reproduce

just use qwik new.

System Info

System:
    OS: macOS 12.6.6
    CPU: (8) arm64 Apple M1 Pro
    Memory: 79.23 MB / 16.00 GB
    Shell: 5.8.1 - /bin/zsh
  Binaries:
    Node: 16.14.2 - ~/.nvm/versions/node/v16.14.2/bin/node
    npm: 8.5.0 - ~/.nvm/versions/node/v16.14.2/bin/npm
  Browsers:
    Chrome: 117.0.5938.149
    Safari: 16.5
  npmPackages:
    @builder.io/qwik: ^1.2.13 => 1.2.13 
    @builder.io/qwik-city: ^1.2.13 => 1.2.13 
    undici: 5.22.1 => 5.22.1 
    vite: 4.4.7 => 4.4.7

Additional Information

Sorry to drop such a trivial bug. I am learning qwik for the first time, so submitting the issue with a goal of submitting a PR for it shortly as soon as I get my bearings.

@chug2k chug2k added STATUS-1: needs triage New issue which needs to be triaged TYPE: bug Something isn't working labels Oct 7, 2023
@wmertens
Copy link
Member

wmertens commented Oct 7, 2023

Can you provide more details? How are you creating CamelCaseComponent and how do you import it, where do you notice the problem?

@wmertens
Copy link
Member

@chug2k Do you mean that the qwik new script changes the casing? PR very welcome :-)

@wmertens wmertens added COMP: starters COMMUNITY: PR is welcomed We think it's a good feature to have but would love for the community to help with the PR for it and removed STATUS-1: needs triage New issue which needs to be triaged labels Oct 31, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
COMMUNITY: PR is welcomed We think it's a good feature to have but would love for the community to help with the PR for it COMP: starters TYPE: bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants