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 skin avatars for Players. #3

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

greggubben
Copy link
Contributor

I found a site that will create images from user's skins. I added the images to the Player's pages.

@m0r13
Copy link
Owner

m0r13 commented Jun 28, 2014

Very good idea too!

Though I would rather host the image creation script I am also using for mapcrafter-playermarkers directly on mapcrafter.org. Users with Mapcrafter maps and player markers could also use that hosted script instead of setting up their own one (if that is too much effort for them). Also I think maybe it would be a bit better to show only the heads of the players on the player list.

I will merge your changes, set up my own script and maybe adjust some small things.

@greggubben
Copy link
Contributor Author

If you have a hosted image creation script that would great. Then you can get metrics on how often this script is used. :-) Good idea about using the heads for the player page.

BTW: I like the improvements you made to the death counts.

@m0r13
Copy link
Owner

m0r13 commented Jan 25, 2015

Sorry it took some time, but I finally managed to implement the player avatars! ;)

I wrote a small Django app to generate the player head/body avatars (minavatars) which I host on mapcrafter.org, and integrated them into the template.html file.

@arsenyspb
Copy link

Hi @m0r13 I'm new to the wonderful world of mapcrafter.org so I might not be up to speed, but looks like the latest template.html 49cd072 uses the following:
<img class="avatar" src="http://avatars.mapcrafter.org/head/{{ user.username }}/4" />
...however the host there responds with:
502 Bad Gateway nginx/1.6.2
Could you may be help by checking the service?..

@arsenyspb
Copy link

Hey @m0r13 any chance you can look at the "Bad Gateway" above?.. thanks

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