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

fix: store related errors #2154

Merged
merged 8 commits into from
Feb 5, 2024
Merged

fix: store related errors #2154

merged 8 commits into from
Feb 5, 2024

Conversation

dudantas
Copy link
Member

Resolves #2153

Fixed cask and kegs
Fixed stackable offers check store inbox limit and others fixes.
Added new config keys for store inbox and loot pouch max limit.
@dudantas

This comment was marked as resolved.

@eightbites

This comment was marked as resolved.

@dudantas

This comment was marked as resolved.

@eightbites
Copy link

amigo nao é assim o problema, o problema é quando tu ta com o gold pouch na inbox e ela ta cheia de gold as 63 pages esta lotada e tu tenta compra algo na store voce vera que da canceled, da forma que tu ta testando ai esta funcionando normal.... mais da forma que expliquei agora vc entendera o motivo do bug, faz o teste da forma que falei encha sua gold pouch de coins ate chega 63 pages e depois tente compra item na store

@cmcpro7
Copy link

cmcpro7 commented Jan 28, 2024

Boa tarde, não consigo usar o transferable coins para comprar item na store, é como se ele não tivesse utilidade

@jacques-tony
Copy link

Não estou testando esse pr em especifico, mas tinha sinalizado esse mesmo problema ao comprar com a loot pouch cheia aqui nesse issue , quando fiz o merge do 13.32 para testar a pr da solução: essa aqui continuou com o mesmo problema.

@eightbites
Copy link

@jacques-tony poise, tanto o canary-main quanto esse PR ta com esse problema ainda, a solução de capacidade max nao resolveu esse problema que temos sobre a gold pouch lotada.

@dudantas
Copy link
Member Author

amigo nao é assim o problema, o problema é quando tu ta com o gold pouch na inbox e ela ta cheia de gold as 63 pages esta lotada e tu tenta compra algo na store voce vera que da canceled, da forma que tu ta testando ai esta funcionando normal.... mais da forma que expliquei agora vc entendera o motivo do bug, faz o teste da forma que falei encha sua gold pouch de coins ate chega 63 pages e depois tente compra item na store

Certo, mas você não explicou isso antes. Você só falou que não estava funcionando, precisa dar detalhes quando for reportar. Vou fazer esse teste ai e arrumar, aí aviso para vocês testarem.

@eightbites
Copy link

ok, mais quando abrir o issue eu falei bem explicado rsrsr acho que tu nao viu la, mais vamos pra cima...

@eightbites
Copy link

caso queira testa rapido, basta voce adiciona pra algum monstro dropa o item crystal coin com chance alta e com uma quantidade alta, e coloca para os crystal coin ir pra gold pouch ao clica no monstro.... assim vc consegue chega na page 63 rapidamente

@dudantas
Copy link
Member Author

caso queira testa rapido, basta voce adiciona pra algum monstro dropa o item crystal coin com chance alta e com uma quantidade alta, e coloca para os crystal coin ir pra gold pouch ao clica no monstro.... assim vc consegue chega na page 63 rapidamente

Foi por isso que eu adicionei no config.lua o limite, fica mais fácil de testar. Assim que eu testar aviso. Acho que já entendi o motivo do bug.

@eightbites
Copy link

@dudantas alguma novidade?

@dudantas
Copy link
Member Author

dudantas commented Jan 31, 2024

@dudantas alguma novidade?

@eightbites, agradeço pela sua paciência e pelo acompanhamento do pull request. Enviei o commit com a correção.

Se você valoriza o trabalho que estamos realizando e deseja apoiar o desenvolvimento contínuo do projeto, considere fazer uma doação através do nosso link. Sua contribuição com os testes é muito apreciada e nos ajuda a manter o projeto ativo e em constante evolução.

Obrigado novamente pelo seu apoio e interesse no projeto!

Com a correção que fiz neste último commit, algumas mudanças significativas vão ocorrer:
• No lugar de "gold pouch" vai se chamar "loot pouch" se no config.lua estiver ativado para o loot dos monstros irem para ela.
• Na descrição, vai constar mais detalhes, como por exemplo a quantidade de slots disponíveis (o meu está 10 por que eu diminui para fins de testes).
image
• Ao comprar e se não tiver slot, vai informar a quantidade de slots disponíveis e a quantidade necessária:
image

@dudantas dudantas added this pull request to the merge queue Feb 5, 2024
Copy link

sonarqubecloud bot commented Feb 5, 2024

Quality Gate Failed Quality Gate failed

Failed conditions

D Maintainability Rating on New Code (required ≥ A)

See analysis details on SonarCloud

idea Catch issues before they fail your Quality Gate with our IDE extension SonarLint SonarLint

@dudantas dudantas removed this pull request from the merge queue due to a manual request Feb 5, 2024
@dudantas dudantas added this pull request to the merge queue Feb 5, 2024
Merged via the queue into main with commit 428acba Feb 5, 2024
48 checks passed
@dudantas dudantas deleted the dudantas/fix-store-errors branch February 5, 2024 14:48
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.

Bug Store
6 participants