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

Update Vendor.cs, Resolves #1126 and Resolves #811 #1140

Merged
merged 1 commit into from
Dec 15, 2018
Merged

Update Vendor.cs, Resolves #1126 and Resolves #811 #1140

merged 1 commit into from
Dec 15, 2018

Conversation

dgatewood
Copy link
Collaborator

Fixes issue where vendors gave you X times the value for an amount of items in a stack that you were selling.

@dgatewood
Update Vendor.cs
110cfb7 
Fixes issue where vendors gave you X times the value for an amount of items in a stack that you were selling.
mcreedjr
mcreedjr reviewed Dec 14, 2018
View reviewed changes
Copy link
Contributor

@mcreedjr mcreedjr left a comment

Choose a reason for hiding this comment

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

Might I suggest referencing the open issue that this PR addresses? It will then get auto closed once your PR is merged.

@LtRipley36706 LtRipley36706 changed the title Update Vendor.cs Update Vendor.cs, Resolves #1126 Dec 15, 2018
@LtRipley36706
Copy link
Member

LtRipley36706 commented Dec 15, 2018 •
edited

Resolves #1126
Resolves #811

@LtRipley36706 LtRipley36706 changed the title Update Vendor.cs, Resolves #1126 Update Vendor.cs, Resolves #1126 Resolves #811 Dec 15, 2018
@LtRipley36706 LtRipley36706 changed the title Update Vendor.cs, Resolves #1126 Resolves #811 Update Vendor.cs, Resolves #1126 and Resolves #811 Dec 15, 2018
@LtRipley36706 LtRipley36706 merged commit ffa1a9c into ACEmulator:master Dec 15, 2018
@dgatewood dgatewood deleted the fixVenderBuying branch December 17, 2018 02:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mcreedjr mcreedjr mcreedjr left review comments

@LtRipley36706 LtRipley36706 LtRipley36706 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@dgatewood @LtRipley36706 @mcreedjr