Skip to content

Conversation

@kostysh
Copy link
Contributor

@kostysh kostysh commented Jun 7, 2022

No description provided.

Galeaf11 and others added 30 commits June 6, 2022 11:09
This standards the storage of the Availability protobuf message.
Particularly when being called, it just simply wraps the leveldb
database. Functionally, this is required as a per day override may be
used in order to 'ram' the availability of a space type to 0 when a
guesthouse wants to 'block' out a specific space type for a day.

BREAKING CHANGE: 🧨 Discrete daily override does NOT only increment numSpaces
Updates the storage schema to add missing rates for otherItems, as well
as convering on standards for space and otherItems with respect to
metadata keys.

BREAKING CHANGE: 🧨 metadata keys for spaces are changed

✅ Closes: #13
GET: /facility/:facilityId/modifier/:modifierKey; GET:
/facility/:facilityId/:itemKey/:itemId/modifier/:modifierKey

✅ Closes: #18
Errors that means 404 in repositories now are ApiError.NotFound
POST: /facility/:facilityId/modifier/:modifierKey

✅ Closes: #18
POST: /facility/:facilityId/:itemKey/:itemId/modifier/:modifierKey

✅ Closes: #18
DELETE: /facility/:facilityId/modifier/:modifierKey; DELETE:
/facility/:facilityId/:itemKey/:itemId/modifier/:modifierKey

✅ Closes: #18
Optimised repository initialisation with ItemModifierRepository

✅ Closes: #18
@kostysh kostysh requested a review from mfw78 June 7, 2022 10:34
Copy link
Contributor

@mfw78 mfw78 left a comment

Choose a reason for hiding this comment

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

LGTM

@mfw78 mfw78 merged commit 1c500d0 into main Jun 7, 2022
@github-actions
Copy link

github-actions bot commented Jun 7, 2022

🎉 This PR is included in version 2.0.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@mfw78 mfw78 deleted the feat/leveldb branch June 7, 2022 11:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants