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.

Sign up for GitHub

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

Jump to bottom

default images to variant[0], or parent variant #18

Closed
aaronjudd opened this issue Apr 11, 2014 · 1 comment
Closed

default images to variant[0], or parent variant #18

aaronjudd opened this issue Apr 11, 2014 · 1 comment
Assignees
@saarmstrong
Milestone
Alpha

Comments

@aaronjudd
Copy link
Contributor

If variant has no images default to variant[0] image

if child variant has no images, default to parent variant image, else variant [0]
@aaronjudd aaronjudd added this to the Alpha milestone Apr 11, 2014
@aaronjudd aaronjudd self-assigned this Apr 11, 2014
@saarmstrong saarmstrong assigned saarmstrong and unassigned aaronjudd May 4, 2014
@saarmstrong
Copy link
Contributor

  • 78ff9636fc5b1089376fa9c6976f8a8ab9439f5d Default variant's image to the first variant's image if current variant lacks an image.
  • 50e5eec897f2aa7c23da6a0902252f1e479d19a0 updated var name ; check for admin role before showing primary variant's images if none assigned

cmbirk pushed a commit to cmbirk/reaction that referenced this issue Aug 18, 2019
@samkelleher
Merge pull request reactioncommerce#18 from sportsdirect/feat-machiko…
dab1079 
…-update-to-develop

feat(tag manager ui, refactors): add Tag Manager UI, update to latest develop
@snyk-bot snyk-bot mentioned this issue Aug 4, 2021
Open
This was referenced Aug 19, 2021
Merged
Closed
Closed
Closed
This was referenced Sep 21, 2021
Merged
Closed
Closed
Merged
@snyk-bot snyk-bot mentioned this issue Sep 29, 2021
Closed
This was referenced Mar 2, 2022
Closed
Closed
@snyk-bot snyk-bot mentioned this issue Sep 18, 2022
Open
zenweasel pushed a commit that referenced this issue Sep 20, 2022
@Akarshit
Merge pull request #18 from reactioncommerce/dependabot/npm_and_yarn/…
f0a6e84 
…ws-7.4.6

chore(deps): Bump ws from 7.3.0 to 7.4.6
zenweasel pushed a commit that referenced this issue Sep 20, 2022
@Akarshit
Merge pull request #18 from reactioncommerce/dependabot/npm_and_yarn/…
ede9480 
…glob-parent-5.1.2

chore(deps): Bump glob-parent from 5.1.1 to 5.1.2
zenweasel pushed a commit that referenced this issue Sep 20, 2022
@Akarshit
Merge pull request #18 from reactioncommerce/dependabot/npm_and_yarn/…
ae417cd 
…glob-parent-5.1.2

chore(deps): Bump glob-parent from 5.1.1 to 5.1.2
zenweasel pushed a commit that referenced this issue Sep 20, 2022
@Akarshit
Merge pull request #18 from reactioncommerce/snyk-upgrade-be91bfd0a39…
c2723f6 
…d1fa39b8809822be27886

[Snyk] Upgrade simpl-schema from 1.10.2 to 1.12.0
zenweasel pushed a commit that referenced this issue Sep 20, 2022
@Akarshit
Merge pull request #18 from reactioncommerce/snyk-upgrade-80ed4976c67…
3733411 
…1dceabed6d4e268cedb01

[Snyk] Upgrade simpl-schema from 1.10.2 to 1.12.0
zenweasel pushed a commit that referenced this issue Sep 20, 2022
@Akarshit
Merge pull request #18 from reactioncommerce/snyk-upgrade-7be334c2609…
4b74ceb 
…83d26b149f2c463d9971d

[Snyk] Upgrade @reactioncommerce/api-utils from 1.16.0 to 1.16.1
zenweasel pushed a commit that referenced this issue Sep 20, 2022
@Akarshit
Merge pull request #18 from reactioncommerce/dependabot/npm_and_yarn/…
7be669a 
…ini-1.3.8

chore(deps): bump ini from 1.3.5 to 1.3.8
zenweasel pushed a commit that referenced this issue Sep 20, 2022
@Akarshit
Merge pull request #18 from reactioncommerce/dependabot/npm_and_yarn/…
ad91d96 
…browserslist-4.16.7

chore(deps): bump browserslist from 4.12.0 to 4.16.7
zenweasel pushed a commit that referenced this issue Sep 20, 2022
@Akarshit
Merge pull request #18 from reactioncommerce/dependabot/npm_and_yarn/…
197c50e 
…handlebars-4.7.7

chore(deps): bump handlebars from 4.7.6 to 4.7.7
zenweasel pushed a commit that referenced this issue Sep 20, 2022
@Akarshit
Merge pull request #18 from reactioncommerce/snyk-upgrade-af03d855e1f…
d50e99c 
…66d4301304b97352a1468

[Snyk] Upgrade @reactioncommerce/api-utils from 1.14.2 to 1.16.5
zenweasel pushed a commit that referenced this issue Sep 20, 2022
@Akarshit
Merge pull request #18 from reactioncommerce/akarshit-update-lodash
8bf175a 
fix: use shrinkwrap to fix lodash on publish
zenweasel pushed a commit that referenced this issue Sep 20, 2022
@delagroove
Merge pull request #18 from reactioncommerce/snyk-upgrade-cdc99cca4d5…
3eee14b 
…6810b31ee3b4c0442658e

[Snyk] Upgrade @reactioncommerce/api-utils from 1.16.5 to 1.16.7
zenweasel pushed a commit that referenced this issue Sep 20, 2022
@delagroove
Merge pull request #18 from reactioncommerce/snyk-upgrade-fe7cdaebf94…
dcd6f4e 
…e9d30e6a86e4d3531a45a

[Snyk] Upgrade @reactioncommerce/api-utils from 1.16.7 to 1.16.9
@snyk-bot snyk-bot mentioned this issue Sep 27, 2022
Open
@snyk-bot snyk-bot mentioned this issue Oct 6, 2022
Open
@csnyk csnyk mentioned this issue Oct 7, 2022
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@saarmstrong saarmstrong

Labels
None yet
Projects
None yet
Milestone
Alpha
Development

No branches or pull requests
2 participants
@saarmstrong @aaronjudd