Giter Site home page Giter Site logo

Comments (10)

gioboa avatar gioboa commented on August 14, 2024 1

Thanks for your report, we will work to clarify and create FAQs

from storefront-qwik-starter.

msj121 avatar msj121 commented on August 14, 2024 1

@gioboa Thanks - I didn't mean to be rude, I'd be happy to help if you point me in some right directions.

from storefront-qwik-starter.

msj121 avatar msj121 commented on August 14, 2024 1

For 2. It was just the following:

https://github.com/vendure-ecommerce/storefront-qwik-starter/blob/9099a34b8e8eb5677e435e3c369a8ce659cc82d7/src/routes/sign-up/index.tsx#L65C2-L71C12

<p>
      Account registration is not supported by the demo Vendure instance. In order to use
      it, please connect to your own local / production instance.
</p>

This misled me into thinking my storefront was connecting to the demo vendure, when after changing .env it was actually connected to my own. Perhaps, making this message dependent on the .env url being the demo vendure url, or perhaps just rewriting to say:

If this is still using public vendure instance, you cannot create an account. 
Please connect your own vendure instance by modifying the .env file to point to your instance.

Or something like that. Not a big deal, but it did throw me for a little bit, I should've checked the code, but I didn't realize it was a hard-coded message since it states matter of fact that I needed to create my own instance.

from storefront-qwik-starter.

gioboa avatar gioboa commented on August 14, 2024 1

We can log a message in the terminal and create a FAQ for that. Would you like to open a PR to add FAQs in the readme?

from storefront-qwik-starter.

ahnpnl avatar ahnpnl commented on August 14, 2024 1

It would be nice to have architecture things documented too. For example, Iā€™m looking at this repo as an example of how to use graphql because Iā€™m using a client lib like Apollo or urql.

The architecture decisions can be:

  • App structure
  • Library decisions
  • Component design
  • When to use this or that and explanation
  • Best practices: SEO, api interactions.
    and so on :)

from storefront-qwik-starter.

gioboa avatar gioboa commented on August 14, 2024

@msj121 no problem. Can you list here the questions you would like to clarify?

from storefront-qwik-starter.

msj121 avatar msj121 commented on August 14, 2024

Okay, after doing a lot more reading on Qwik City specifically and cloudfare things are starting to make a lot more sense... Once I get things running I will try to contribute something to documentation, either here or somewhere else for anyone who needs some advice and is new to everything.

1). Out of curiosity, is the ecommerce store tailored for cloudfare (ie: is there something that makes the adapter for cloudfare necessary) or can express or some other adapter be used?

2). Sign up page, by default states that the application is not connected to your own vendure api (ie: Message about cannot setup user). This confused me even after changing, .env file to indeed connect, I thought it didn't work. Perhaps is there a way to make the message more universal, so it can work on demo but also a git clone. Like using the word "ignore if local" or something? Not a big deal, but would be useful.

from storefront-qwik-starter.

gioboa avatar gioboa commented on August 14, 2024

Great, you can contribute by adding few FAQs in the Readme. This cloud be valuable

  1. you can deploy the application in different environment (e.g vercel, fastify, etc. etc) there isn't a specific cloudflare feature for this application

  2. what do you mean with message?

from storefront-qwik-starter.

gioboa avatar gioboa commented on August 14, 2024

I added the first FAQ question
https://github.com/vendure-ecommerce/storefront-qwik-starter#can-i-host-the-application-in-my-environment-or-is-limited-to-cloudflare
@msj121 I really would like to have your point of view because I'm biased.
Pls drop a PR to add more questions

from storefront-qwik-starter.

gioboa avatar gioboa commented on August 14, 2024

@ahnpnl yep, sounds reasonable

from storefront-qwik-starter.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    šŸ–– Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. šŸ“ŠšŸ“ˆšŸŽ‰

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ā¤ļø Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.