Skip to content

Bug(Shops): Round up shop purchase quantity#829

Merged
itinerare merged 1 commit into
lk-arpg:mainfrom
AW0005:bug-fix/shop_quantity
Feb 11, 2024
Merged

Bug(Shops): Round up shop purchase quantity#829
itinerare merged 1 commit into
lk-arpg:mainfrom
AW0005:bug-fix/shop_quantity

Conversation

@AW0005
Copy link
Copy Markdown
Contributor

@AW0005 AW0005 commented Feb 9, 2024

Because otherwise 0.5 because a whole item at half price.

@itinerare itinerare added bug Something isn't working needs review Pull requests that are pending community review labels Feb 9, 2024
Copy link
Copy Markdown
Contributor

@SpeedyD SpeedyD left a comment

Choose a reason for hiding this comment

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

Seems good to me. But should this not be in release, rather than main??

@itinerare
Copy link
Copy Markdown
Member

itinerare commented Feb 10, 2024

No I ok'd it, it's all good.
That said, I'll merge tomorrow.

@itinerare itinerare added reviewed Pull requests that have received community review and are pending merge and removed needs review Pull requests that are pending community review labels Feb 10, 2024
@SpeedyD
Copy link
Copy Markdown
Contributor

SpeedyD commented Feb 10, 2024

I had no problem with it regardless. 👍

@itinerare itinerare merged commit 2b11217 into lk-arpg:main Feb 11, 2024
AW0005 added a commit to AW0005/lorekeeper-extensions that referenced this pull request Jun 29, 2024
* Fix(MYOs): Allows character images to work locally

* fix(comments): more robust handling for non user-user comments

* fix(comments): adjust staff-user default handling

* fix(comments): remove empty staff-staff default

* fix(items): disable box button on submit (lk-arpg#710)

* fix(sales): allow setting minimum for ota and xta (lk-arpg#702)

- Fixes missing minimum options for ota and xta
- Also adds consistent stylization for ota and xta alongside pwyw

* chore(update): 2.1.3

* hotfix(shops): round up shop purchase quantity (lk-arpg#829)

* fix(shops): validate quantity against purchase limit
- bump version

* hotfix(security|requests): add middleware that parses post fields (lk-arpg#974)

* hotfix(security|requests): add middleware that parses post fields

* fix: handle arrays

* fix: add "$this->" prefix to function calls

* chore(update): update version

* fix: add numeric check to parse to prevent dropdowns from being parsed (lk-arpg#976)

* hotfix(security|requests): add middleware that parses post fields

* fix: handle arrays

* fix: add "$this->" prefix to function calls

* fix(requests): prevent numeric value from being parsed

* chore(update): update version

---------

Co-authored-by: Mercury <63693594+itinerare@users.noreply.github.com>

* add default criteria and currency selection

---------

Co-authored-by: SpeedyD <speedydvv@gmail.com>
Co-authored-by: itinerare <ad@itinerare.net>
Co-authored-by: itinerare <63693594+itinerare@users.noreply.github.com>
Co-authored-by: Newt <dragonminer101@gmail.com>
Co-authored-by: moif <104588768+AW0005@users.noreply.github.com>
preimpression added a commit to preimpression/lorekeeper that referenced this pull request Aug 11, 2024
* Fix(MYOs): Allows character images to work locally

* fix(comments): more robust handling for non user-user comments

* fix(comments): adjust staff-user default handling

* fix(comments): remove empty staff-staff default

* fix(items): disable box button on submit (lk-arpg#710)

* fix(sales): allow setting minimum for ota and xta (lk-arpg#702)

- Fixes missing minimum options for ota and xta
- Also adds consistent stylization for ota and xta alongside pwyw

* chore(update): 2.1.3

* hotfix(shops): round up shop purchase quantity (lk-arpg#829)

* fix(shops): validate quantity against purchase limit
- bump version

* hotfix(security|requests): add middleware that parses post fields (lk-arpg#974)

* hotfix(security|requests): add middleware that parses post fields

* fix: handle arrays

* fix: add "$this->" prefix to function calls

* chore(update): update version

* fix: add numeric check to parse to prevent dropdowns from being parsed (lk-arpg#976)

* hotfix(security|requests): add middleware that parses post fields

* fix: handle arrays

* fix: add "$this->" prefix to function calls

* fix(requests): prevent numeric value from being parsed

* chore(update): update version

---------

Co-authored-by: Mercury <63693594+itinerare@users.noreply.github.com>

---------

Co-authored-by: SpeedyD <speedydvv@gmail.com>
Co-authored-by: itinerare <ad@itinerare.net>
Co-authored-by: itinerare <63693594+itinerare@users.noreply.github.com>
Co-authored-by: Newt <dragonminer101@gmail.com>
Co-authored-by: moif <104588768+AW0005@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working reviewed Pull requests that have received community review and are pending merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants