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.

Sign up for GitHub

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

Jump to bottom

[Issue #6684] GNIP-78: GeoNode generic "Apps" model to include pluggable entities into the framework #6713

Merged
merged 311 commits into from
Dec 10, 2020
Merged

[Issue #6684] GNIP-78: GeoNode generic "Apps" model to include pluggable entities into the framework #6713

merged 311 commits into from
Dec 10, 2020

Conversation

afabiani
Copy link
Member

Closes #6684

Checklist

Reviewing is a process done by project maintainers, mostly on a volunteer basis. We try to keep the overhead as small as possible and appreciate if you help us to do so by completing the following items. Feel free to ask in a comment if you have troubles with any of them.

For all pull requests:

  • Confirm you have read the contribution guidelines
  • You have sent a Contribution Licence Agreement (CLA) as necessary (not required for small changes, e.g., fixing typos in the documentation)
  • Make sure the first PR targets the master branch, eventual backports will be managed later. This can be ignored if the PR is fixing an issue that only happens in a specific branch, but not in newer ones.

The following are required only for core and extension modules (they are welcomed, but not required, for contrib modules):

  • There is a ticket in https://github.com/GeoNode/geonode/issues describing the issue/improvement/feature (a notable exemption is, changes not visible to end-users)
  • The issue connected to the PR must have Labels and Milestone assigned
  • PR for bug fixes and small new features are presented as a single commit
  • Commit message must be in the form "[Fixes #<issue_number>] Title of the Issue"
  • New unit tests have been added covering the changes, unless there is an explanation on why the tests are not necessary/implemented
  • This PR passes all existing unit tests (test results will be reported by travis-ci after opening this PR)
  • This PR passes the QA checks: flake8 geonode
  • Commits changing the settings, UI, existing user workflows, or adding new functionality, need to include documentation updates

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.

afabiani and others added 30 commits October 12, 2020 09:29
[Hardening] - Recenet Activity List for Documents error when actor is…
f5ba15a 
… None
[Frontend] Monitoring: Bump "node-sass" to version 4.14.1
9a91223
[Frontend] Bump jquery to version 3.5.1
5220754
Merge branch 'master' of https://github.com/GeoNode/geonode into rest…
9c5ba0d 
…_api_v2_proof_of_concept
Merge branch 'master' of https://github.com/GeoNode/geonode into rest…
9dc4e74 
…_api_v2_proof_of_concept
@t-book
[Fixes: #6519] Bump jquery to 3.5.1 (#6526)
ab040b5 
(cherry picked from commit e532813)

# Conflicts:
#	geonode/static/lib/css/assets.min.css
#	geonode/static/lib/css/bootstrap-select.css
#	geonode/static/lib/css/bootstrap-table.css
#	geonode/static/lib/js/assets.min.js
#	geonode/static/lib/js/bootstrap-select.js
#	geonode/static/lib/js/bootstrap-table.js
#	geonode/static/lib/js/leaflet-plugins.min.js
#	geonode/static/lib/js/leaflet.js
#	geonode/static/lib/js/moment-timezone-with-data.js
#	geonode/static/lib/js/underscore.js
Merge branch 'master' of https://github.com/GeoNode/geonode into rest…
f25eac3 
…_api_v2_proof_of_concept

# Please enter a commit message to explain why this merge is necessary,
# especially if it merges an updated upstream into a topic branch.
#
# Lines starting with '#' will be ignored, and an empty message aborts
# the commit.
Merge branch 'master' of https://github.com/GeoNode/geonode into rest…
fd078fe 
…_api_v2_proof_of_concept
[Hardening] Re-create the map thumbnail only if it is missing
584690b
Merge branch 'master' of https://github.com/GeoNode/geonode into rest…
30366aa 
…_api_v2_proof_of_concept
Merge branch 'master' of https://github.com/GeoNode/geonode into rest…
3fb9707 
…_api_v2_proof_of_concept
Fixes error with GDAL 3.0.4 due to a breaking change on GDAL (https:/…
bd6ed2d 
…/code.djangoproject.com/ticket/30645)
Fixes error with GDAL 3.0.4 due to a breaking change on GDAL (https:/…
656eedc 
…/code.djangoproject.com/ticket/30645)
Merge branch 'master' of https://github.com/GeoNode/geonode into rest…
5066c36 
…_api_v2_proof_of_concept
- Introducing the concept of "GeoNode App" Resource Base
509bea9
[GeoApps] Add "Create new" Button to the apps list page
9e860d4
[GeoApps] Hooking Resources List pages
f0c52b3
[GeoApps] Hooking GeoApp List page
2df4fa7
[GeoApps] Hooking GeoApp rest v2 API serializers fixes
3adb9e7
[GeoApps] Fix resourcebase_api polymorphic ctype filter
93e1cb3
[GeoApps] REST API v2 "geostories" endpoints optimizations
f4d4591
Merge branch 'search-by-extent' of https://github.com/mtnorthcott/geo…
a44d1c9 
…node into rest_api_v2_geonode_apps
[REST APIs V2] Make use of the new "bbox_polygon" field based on GeoD…
0120094 
…jango
Merge branch 'master' of https://github.com/GeoNode/geonode into rest…
0e30277 
…_api_v2_geonode_apps
[Fixes RemoteServices bbox parse] Merge branch 'search-by-extent' of h…
b0f3301 
…ttps://github.com/mtnorthcott/geonode
Merge branch 'master' of https://github.com/GeoNode/geonode into rest…
91052cf 
…_api_v2_geonode_apps
[Fix migrations] Merge branch 'search-by-extent' of https://github.co…
2e76217 
…m/mtnorthcott/geonode
[Fix migrations] Merge branch 'search-by-extent' of https://github.co…
547b256 
…m/mtnorthcott/geonode
[GeoApps] Adding "geoapp_edit" page
2ec06a2
[GeoApps] Adding "geoapp_edit" page context
8f6e3db
afabiani and others added 17 commits November 29, 2020 12:34
Merge branch 'master' of https://github.com/GeoNode/geonode into rest…
4aad9b7 
…_api_v2_geonode_apps
Merge branch 'master' of https://github.com/GeoNode/geonode into rest…
cd4bdf3 
…_api_v2_geonode_apps
Merge branch 'master' of https://github.com/GeoNode/geonode into rest…
d92f1e7 
…_api_v2_geonode_apps
Merge branch 'master' of https://github.com/GeoNode/geonode into rest…
47121e4 
…_api_v2_geonode_apps
Merge branch 'master' of https://github.com/GeoNode/geonode into rest…
eea5e15 
…_api_v2_geonode_apps
[Hardening] More resiliet to 'missing thumbnail' on filesystem issues
2fcc739
- GeoApp Test Cases
b7bec4a
- Typo
770606e
- Update mapstore client and adapter versions
cd63332
- Set local .sh files exec perms
0ce8d09
Merge branch 'master' of https://github.com/GeoNode/geonode into rest…
2e4eee1 
…_api_v2_geonode_apps
- Bump pycsw to version 2.6.0
9d5c4b1
- Bump pycsw to version 2.6.0
43227ad
- Bump pycsw to version 2.6.0
1532809
Merge branch 'master' of https://github.com/GeoNode/geonode into rest…
bf09fe1 
…_api_v2_geonode_apps
@afabiani
Merge branch 'master' of https://github.com/GeoNode/geonode into rest…
4456702 
…_api_v2_geonode_apps
@afabiani
- Align "setup.cfg" to "requirements.txt"
3f9e7dc
@afabiani afabiani self-assigned this Dec 10, 2020
@cla-bot cla-bot bot added the cla-signed CLA Bot: community license agreement signed label Dec 10, 2020
@lgtm-com
Copy link

lgtm-com bot commented Dec 10, 2020

This pull request introduces 3 alerts when merging 3f9e7dc into 6527b05 - view on LGTM.com

new alerts:

  • 2 for Unused local variable
  • 1 for Unused import

giohappy
giohappy previously approved these changes Dec 10, 2020
View reviewed changes
@afabiani afabiani merged commit f554424 into master Dec 10, 2020
@afabiani afabiani deleted the rest_api_v2_geonode_apps branch December 10, 2020 16:00
@lgtm-com
Copy link

lgtm-com bot commented Dec 10, 2020

This pull request introduces 3 alerts when merging e10f843 into 6527b05 - view on LGTM.com

new alerts:

  • 2 for Unused local variable
  • 1 for Unused import

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@giohappy giohappy giohappy left review comments

@t-book t-book Awaiting requested review from t-book

@gannebamm gannebamm Awaiting requested review from gannebamm

@francbartoli francbartoli Awaiting requested review from francbartoli

Assignees

@afabiani afabiani

Labels
cla-signed CLA Bot: community license agreement signed
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

GNIP-78: GeoNode generic "Apps" model to include pluggable entities into the framework
5 participants
@afabiani @giohappy @t-book @petrus7 @gannebamm