Feature/stock market mode #369
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
v2023.9.1 release
Charge endpoint now checks if 'stock-market-mode' is enabled, which calculates price of a product based on the purchase history
Gotta stick to conventions yo
Feature/stock market price query
* Remove unused mutation for proudcts and product orders * Upgrade graphene-django-cud to 0.11.1 * Add addict as dev-dependency (when writing schema tests the gql decorator cannot find the user if its not in an object. Meaning it would always give permission errors) * Write basic schema tests for new endpoints * Add `fail_silently` handlingt to stock price calculation util function * Add UserWithPermission factory
Due to improper use of timezone aware objects the datetimes passed down to the helper function would default to a +01.22 timezone instead of the +01.00 which is expected from the configuration of this project
…d-testing feat(economy): ghost product mutation, tests and other misc.
Christ this is in no way an efficient way to do it. But I think it works
feat(economy): stock market mode price history resolver
|
Add more tests for the stock price calculation. after adding the time window delta there are probably more edge cases to test for |
* Add model representing market crashes * Add another filter to stock price calculation that filters on latest market crash if it exists * Add crash market mutation * Add latest market crash query
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This will be a longer running feature branch. The idea is to add the following functionality
Must haves:
Nice to haves: