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: don't send inbox store item to market #1859

Merged

Conversation

MUN1Z
Copy link
Contributor

@MUN1Z MUN1Z commented Nov 20, 2023

Description

Adjustments to dont send inbox store item to market in client

Behaviour

Actual

O servidor está enviando para o Market do cliente informações de itens que são do Store Inbox.

adbda34a685f8a7c1baa959534440b22.mp4

Expected

Não deve ser possível visualizar os itens de Store Inbox no Market do cliente.

ec351d812cc2f0d3396eeb7f2f0318f6.mp4

Fixes #issuenumber

Type of change

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

How Has This Been Tested

Comprar poções de mana na Store ou pega-las da recompensa diaria e adicionar no deposito.
Abrir o Market e pesquisar por "mana".

Checklist

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I checked the PR checks reports

Copy link

sonarcloud bot commented Nov 21, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

0.0% 0.0% Coverage
0.0% 0.0% Duplication

@dudantas dudantas merged commit 71bdb2f into opentibiabr:main Nov 21, 2023
25 checks passed
@dudantas dudantas changed the title fix: adjustments to dont send inbox store item to market fix: don't send inbox store item to market Nov 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants