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.

Sign up for GitHub

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

Jump to bottom

Trading Villager's Settings Error #788

Closed
Typheonn opened this issue Jun 25, 2019 · 1 comment
Closed

Trading Villager's Settings Error #788

Typheonn opened this issue Jun 25, 2019 · 1 comment
Assignees
@tastybento
Labels
Status: Done This issue has been completed or answered. This pull request has been merged. Type: Bug
Milestone
1.6.0

Comments

@Typheonn
Copy link

Description
When you enable Trading with villager but not Containers, it's impossible to trade with villagers in islands. We can ses that message : "Ile protégée : Container access disabled"

Steps to reproduce the behavior:

  1. Go to an other islands (not your island) with villager and where settings allow trades with villager but not containers access.
    2 . Click on one item to make trade.

Server Information:

  • Database being used : yml
  • BentoBox version: 1.5.3
  • Addons installed?
    BSkyBlock : 1.5.0
    Biomes: 1.5.0.0
    Challenges: 0.8.0-SNAPSHOT-Entity limits #278
    Level: 1.5.0
    WelcomeWarps: 1.5.0
@Poslovitch Poslovitch transferred this issue from BentoBoxWorld/BSkyBlock Jun 25, 2019
@tastybento tastybento self-assigned this Jun 27, 2019
@tastybento tastybento added Status: Pending Waiting for a developer to start working on this issue. Type: Bug labels Jun 27, 2019
@tastybento tastybento added this to the 1.6.0 milestone Jun 27, 2019
@tastybento tastybento removed the Status: Pending Waiting for a developer to start working on this issue. label Jun 27, 2019
tastybento added a commit that referenced this issue Jun 27, 2019
@tastybento
Fixes bug where NPC inventories were protected by the container flag
1f4fa0e 
#788
@tastybento
Copy link
Member

Thanks for the report.

@Poslovitch Poslovitch added the Status: Done This issue has been completed or answered. This pull request has been merged. label Jun 27, 2019
@BONNe BONNe mentioned this issue Jul 1, 2019
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@tastybento tastybento

Labels
Status: Done This issue has been completed or answered. This pull request has been merged. Type: Bug
Projects
None yet
Milestone
1.6.0
Development

No branches or pull requests
3 participants
@tastybento @Poslovitch @Typheonn