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

Added thanksgiving native headdress to avatar builder #2852

Merged
merged 1 commit into from
Nov 15, 2018

Conversation

jakerockland
Copy link
Contributor

@jakerockland jakerockland commented Nov 15, 2018

Description

Added headdress accessory to avatar creator 🦃.

Checklist
  • linter status: 100% pass
  • changes don't break existing behavior
  • commit message follows commit guidelines
Affected core subsystem(s)

avatar

Testing

Tested locally, screenshots attached:

screenshot 2018-11-15 15 21 55

screenshot 2018-11-15 15 22 02

Refers/Fixes

Refs: #2684

@jakerockland jakerockland mentioned this pull request Nov 15, 2018
3 tasks
@owocki owocki merged commit a93348f into gitcoinco:master Nov 15, 2018
@jakerockland jakerockland deleted the thanksgiving-headdress branch November 15, 2018 22:40
@mbeacom
Copy link
Contributor

mbeacom commented Nov 15, 2018

Not sure how I feel about the optics of this avatar item. 🤔

@owocki
Copy link
Contributor

owocki commented Nov 15, 2018

thats a fair point; i know that the intent is to enable folks to celebrate thanksgiving here in the US..

@owocki
Copy link
Contributor

owocki commented Nov 15, 2018

we do have multiple skin colors / gender specific items in the avatar builder. maybe this item is questionable because its a specific culture's artifact?

@jakerockland
Copy link
Contributor Author

jakerockland commented Nov 15, 2018

@mbeacom Like the design of it or like the optics of having a headdress accessory? I agree there is definitely a discussion to be had around it.

On one hand I understand that the headdress is often a ceremonial object of religious importance, so perhaps having the accessory is in poor taste.

On the other hand, the indigenous people who helped the pilgrims learn to farm were invaluable to the early settlers and are fundamental to why we celebrate thanksgiving... I think.

So yeah I'm not sure. I certainly won't have any problem if you guys want to revert/kill this one, but also am not sure what the best way is acknowledging the aspect of the native American involvement/history that we celebrate when we celebrate Thanksgiving.

My 2¢

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.

3 participants