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

level 2 incentive #213

Merged
merged 2 commits into from
Jan 7, 2019
Merged

level 2 incentive #213

merged 2 commits into from
Jan 7, 2019

Conversation

amberleyromo
Copy link

Closes #207.

Where a contributor has earned level 1, but not yet level 2, show a progress bar and messaging toward earning level 2.

screen shot 2019-01-06 at 11 52 30 am

@amberleyromo amberleyromo changed the base branch from master to next January 6, 2019 17:54
@amberleyromo
Copy link
Author

@greglobinski first time working with the progress element (i think) would love feedback on the vendor stuff, if you're more familiar. I've tested in firefox, chrome and safari.

Copy link
Contributor

@jlengstorf jlengstorf left a comment

Choose a reason for hiding this comment

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

Looking good! Let's get this in.

<>
<CodeBadgeBox key={`HOLYBUCKETS`}>
<CodeBadge>
<Name code={`HOLYBUCKETS`}>{`Level 2 Swag Code`}</Name>
Copy link
Contributor

Choose a reason for hiding this comment

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

We should extract this out into the env or something. As this gets more complex we're going to want to consolidate the definitions into a single place or else maintaining this will be a find-and-replace nightmare.

Copy link
Author

Choose a reason for hiding this comment

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

100%. I figured this fell under the category of what we talked about on Slack -- right now complexity is limited to one category of contribution, but a lot of this will have to be extended when we expand incentives 👍

@jlengstorf jlengstorf merged commit a0a802e into next Jan 7, 2019
@jlengstorf jlengstorf deleted the ajr/207-incentive-level-up branch January 7, 2019 22:08
jlengstorf added a commit that referenced this pull request Jan 11, 2019
* chore: add link to the live store

* fix: update callback domain for Auth0

* feat: store redesign (#201)

TODO:
- [x] ~~refactor `ProductImagesBrowser` (for fixing transitions)~~
- [ ] filtering for `ProductListing` (by swag codes) ()
- [ ] internal Ads functionality
- [x] ~~screens transitions/switching on mobile (some fixes/improvements)~~
- [ ] a couple updates in content
- [x] ~~a list of small fixes and improvements in styles~~

Till the moment I fix the bugs in mobile switching/transitions I suggest testing the store on desktop, instead you will be pointing me strange behaviors I am aware of.

* feat: update used swag code badge style (#212)

* feat: style used badges, modify messaging, update badge theming

* camelcase swag code badge title

* fix: revert static test

* feat: level 2 incentive (#213)

Where a contributor has earned level 1, but not yet level 2, show a progress bar and messaging toward earning level 2.

<img width="420" alt="screen shot 2019-01-06 at 11 52 30 am" src="https://user-images.githubusercontent.com/3461087/50739721-b0828600-11a9-11e9-8b0c-98a6c01e3565.png">

closes #207

* feat: add Google Analytics (#216)

* chore: use official Shopify plugin (#217)

- update to the Gatsby-maintained Shopify plugin (see gatsbyjs/gatsby#10955)
- update the Shopify Buy SDK

* fix: keep item quantity in sync (#218)

- adding more of the same item to the cart updates its quantity input
- add a basic “loading” animation while quantities update
- convert the `CartListItem` component to use Hooks

* feat: use custom domain for login

closes #19

* fix: improve cart button accessibility (#220)

* fix: add button label to shopping cart icon

* fix: reorder components to announce shopping cart earlier
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants