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.

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

feat(economy): ghost product mutation, tests and other misc. #371

Merged

Conversation

alexaor
Copy link
Member

@alexaor alexaor commented Oct 31, 2023

Looks like some admission tests are failing. Not sure if this is long-running or not but should probably be fixed before mering this. Looks like github actions need to be fixed

* 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
* Fix broken admission test

	* 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
@alexaor alexaor merged commit b91dc53 into feature/stock-market-mode Nov 2, 2023
@alexaor alexaor deleted the feature/ghost-product-mutation-and-testing branch November 2, 2023 17:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant