-
Notifications
You must be signed in to change notification settings - Fork 189
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
New shop page #2340
New shop page #2340
Conversation
Embeds spreadshop, with some CSS customisation
✅ Deploy Preview for nf-core-main-site ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Will this auto-redirect to the US shop? 🤔 |
Not sure, you tell me! https://deploy-preview-2340--nf-core.netlify.app/shop/ For me it's showing GBP, so it might be a bit funky. I was also thinking about having a FAQ and stuff - so might be worth having our own landing page before going through to the shop pages. Could then have links to the different shop sites, along with some FAQ. What do you think? |
I'm getting pounds 😞 Yeah that could work. I bet we could render one or the other based on location. |
Ok, I googled a bit more and also came to the same conclusion - that the auto redirect wouldn't work. Just put together an index page and then location-specific embed pages under a dropdown. This should be more reliable. Also the FAQ mean that people hopefully know what they're getting themselves into. Let me know what you think. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That looks lovely now! Love the FAQ before a shop renders. I agree that should be more reliable.
One small thing I've noticed is that by selecting the |
Embeds spreadshop, with some CSS customisation
…o shop-attempt-2 # Conflicts: # src/config.ts
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would add a note on the embed page that makes sure to check all pages to see all designs e.g. pipeline specific ones, or mention they exist in the description
Thanks all - I'll add some "highlights" boxes above the button to go to the shop:
With subtext to explain how to eg. see the full list of products. |
Done 🎨 |
Embeds spreadshop, with some CSS customisation
🤫
@netlify /shop