-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
[Fixes #7762] Add keyword to harvested resources #7758
Conversation
Thank you for your pull request and welcome to our community. We require contributors to sign our Contributor License Agreement, and we don't seem to have the users @meomancer on file. In order for us to review and merge your code, please contact the project maintainers to get yourself added. |
This fix geosolutions-it/nexus-geonode#285
|
Codecov Report
@@ Coverage Diff @@
## master #7758 +/- ##
===========================================
+ Coverage 31.67% 56.64% +24.96%
===========================================
Files 686 685 -1
Lines 43630 43604 -26
Branches 5576 5581 +5
===========================================
+ Hits 13820 24699 +10879
+ Misses 29043 17467 -11576
- Partials 767 1438 +671 |
Thank you for your pull request and welcome to our community. We require contributors to sign our Contributor License Agreement, and we don't seem to have the users @meomancer on file. In order for us to review and merge your code, please contact the project maintainers to get yourself added. |
@meomancer the PR looks good to me, but please make sure to update it with the latest changes from |
@afabiani the failing test here seems to be unrelated to the PR, do you agree? |
@ricardogsilva yes correct |
References: #7762
Checklist
For all pull requests:
The following are required only for core and extension modules (they are welcomed, but not required, for contrib modules):
Submitting the PR does not require you to check all items, but by the time it gets merged, they should be either satisfied or inapplicable.