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

OFS-105: created shell backend for 'Calculation' module #1

Merged
merged 5 commits into from
Dec 4, 2020

Conversation

sniedzielski
Copy link
Contributor

In that pull request I want to merge shell backend for 'calculation' module. I implemented models for that module. There are also queries without mutations if it is about the graphQl. Now I can use this model in "ContributionPlan" module as a foreign key after that changes.

There are also some basic tests for creating models.

Copy link
Member

@delcroip delcroip left a comment

Choose a reason for hiding this comment

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

Please add the uuid linked to id in the next PR

@delcroip delcroip merged commit 795960a into develop Dec 4, 2020
@sniedzielski
Copy link
Contributor Author

Ok. I'll add this one in ContributionPlan so as to connect both models thanks to foreign key.

@lgtm-com
Copy link

lgtm-com bot commented Dec 4, 2020

This pull request introduces 10 alerts and fixes 1 when merging ece96cb into ef607d9 - view on LGTM.com

new alerts:

  • 6 for Unused import
  • 2 for Unnecessary pass
  • 2 for 'import *' may pollute namespace

fixed alerts:

  • 1 for Unused import

@sniedzielski sniedzielski deleted the feature/OFS-105 branch March 17, 2021 11:59
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.

None yet

2 participants