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-88: migrating base mutations and delete/replace input types to the Core"module" #25

Merged
merged 6 commits into from
Dec 1, 2020

Conversation

sniedzielski
Copy link
Contributor

@sniedzielski sniedzielski commented Nov 30, 2020

In that pull request I want to include base mutations on "Core" module. It was checked on PolicyHolder module and for that ticket there is another pull request. The link will be below this description. The changes in another modules connected with that issue will be visible during working on them (for example contribution plan, contract etc)

The pull request on "PolicyHolder" module openimis/openimis-be-policyholder_py#4

@lgtm-com
Copy link

lgtm-com bot commented Nov 30, 2020

This pull request introduces 3 alerts when merging 4bf13b5 into 6c54b70 - view on LGTM.com

new alerts:

  • 2 for 'import *' may pollute namespace
  • 1 for Unused import

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 change the folder name of qgl to gql

@dragos-dobre
Copy link
Member

Please solve the alerts from lgtm-com.

@sniedzielski
Copy link
Contributor Author

Ok @delcroip and @dragos-dobre - I've fixed this issue in this repo. I only have to fix on "PolicyHolder" repo with folder name.

@lgtm-com
Copy link

lgtm-com bot commented Dec 1, 2020

This pull request introduces 1 alert when merging ad31d5e into 6c54b70 - view on LGTM.com

new alerts:

  • 1 for 'import *' may pollute namespace

@dragos-dobre dragos-dobre merged commit 563ebee into develop Dec 1, 2020
@dragos-dobre dragos-dobre deleted the feature/OFS-88 branch December 1, 2020 10:19
@lgtm-com
Copy link

lgtm-com bot commented Dec 1, 2020

This pull request introduces 1 alert when merging 239bbdd into 6c54b70 - view on LGTM.com

new alerts:

  • 1 for 'import *' may pollute namespace

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.

3 participants