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

Adiciona conquistas permamentes #125

Merged
merged 10 commits into from
Dec 26, 2018

Conversation

thaismartins
Copy link
Contributor

#46

@thaismartins thaismartins self-assigned this Dec 23, 2018
@thaismartins thaismartins changed the title Adiciona conquistas temporárias Adiciona conquistas permamentes Dec 23, 2018
Copy link
Contributor

@goldblade goldblade left a comment

Choose a reason for hiding this comment

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

Não sei se chegou a pensar nessa lógica @thaismartinsprojects, mas quando vejo essa estrutura na minha cabeça passou a seguinte estrutura:

CAT - PESO
BRONZE - 0
PRATA - 1
OURO - 2
PLATINA - 3
DIAMANTE - 4


pts
1
5
10

se chego a 1 ponto.. verifica qual categoria tenho, se nao tenho pego a mais leve.. viro bronze.. essa logica se aplicaria para as subcategorias tbm

se chego a 5 pontos.. verifico qual categoria tenho.. vou ter bronze.. se cheguei ao peso maior do subnivel mudo para a categoria do proximo pesoCAT - PESO
BRONZE - 0
PRATA - 1
OURO - 2
PLATINA - 3
DIAMANTE - 4

Como o gatilho é quantidade de pontos em determinada função como uma reação enviada por exemplo seguindo essa linha imagino que tenha setado conquistas para as pontuações abaixo:

pts
1
5
10

se chego a 1 ponto.. verifica qual categoria tenho, se nao tenho pego a mais leve.. viro bronze.. essa lógica se aplicaria para as sub-categorias

se chego a 5 pontos.. verifico qual categoria tenho.. vou ter bronze.. se cheguei ao peso maior do sub-nível mudo para a categoria do proximo peso

Precisaria de uma POC para avaliar se essa linha de pensamento teria mais ganhos do que a que foi implementada. Mas é algo para pensarmos.

@@ -0,0 +1,15 @@
reaction:
name: 'Reações'
type: 'reaction'
Copy link
Contributor

Choose a reason for hiding this comment

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

Nesse ponto, senti um pouco redudante o type com a chave reaction. Não poderia usar a chave como referência?

Copy link
Contributor

@lazaroalvarenga lazaroalvarenga left a comment

Choose a reason for hiding this comment

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

config/achievements-network.yml dá pra botar numa pasta pra facilitar a organização dos próximos achievements. tipo config/achievements/network.yml

@rtarantelli rtarantelli merged commit 0a0a631 into impulsoteam:dev Dec 26, 2018
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.

4 participants