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

Upgrade to 4.4.x #58

Open
gannebamm opened this issue Nov 18, 2024 · 24 comments
Open

Upgrade to 4.4.x #58

gannebamm opened this issue Nov 18, 2024 · 24 comments
Assignees
Labels
documentation Improvements or additions to documentation EPIC Maintenance LB 2.1.3 question Further information is requested

Comments

@gannebamm
Copy link
Contributor

With everything (almost) in place, I would like to test upgrading the base GeoNode to 4.4.x branch.

Furthermore, a brief documentation how to upgrade the base GeoNode version shall be given. Where to change what to set the new codebase in place.

@gannebamm gannebamm added documentation Improvements or additions to documentation EPIC Maintenance LB 2.1.3 question Further information is requested labels Nov 18, 2024
@ridoo ridoo self-assigned this Nov 18, 2024
@ridoo
Copy link
Collaborator

ridoo commented Nov 21, 2024

@gannebamm
Copy link
Contributor Author

@ridoo Upgrading to 4.4.x will most likely also resolve this issue: geosolutions-it/geonode-subsites#45 (comment)
right?

@ridoo
Copy link
Collaborator

ridoo commented Nov 27, 2024

Last geonode-subsite release is from Spring 2024, therefore thuenen_atlas includes main as it includes my fix to align with v4.3.x. Once it is merged to main, it will be part of thuenen_atlas. If geosolutions intends to tag or even release current main branch, I will take care of pointing to that version.

@gannebamm
Copy link
Contributor Author

We are currently testing subsite exclusive features with the branch https://github.com/geosolutions-it/geonode-subsites/tree/subsite_exclusive which is included in our test instance as seperated branch (https://github.com/Thuenen-GeoNode-Development/thuenen_atlas/compare/subsite_exclusive_test) and integrated like 4183280

@ridoo
Copy link
Collaborator

ridoo commented Nov 27, 2024

@gannebamm I will install from branch subsite_exclusive which is not merged yet. This will give you the chance to test the whole v4.4.x update on thuenen_4.x (I am mostly through 😅 ).

@ridoo
Copy link
Collaborator

ridoo commented Nov 27, 2024

@gannebamm please review and test: https://github.com/Thuenen-GeoNode-Development/thuenen_atlas/blob/thuenen_4.x

In particular, I aligned thuenen_atlas with the geonode blueprint so that we do not have a non-related history anymore.

A quick exclusive subsite test seemed to work as expected. But please test on your own.

@ridoo
Copy link
Collaborator

ridoo commented Nov 27, 2024

Intiated UPDATE.md (currently more a scratchpad)

@ridoo
Copy link
Collaborator

ridoo commented Nov 29, 2024

detailed update documentation is out of scope. But it should include:

  • why blueprint etc?
  • give high level overview of interdependent repositories
  • big picture needed with brief description

Edit: @hvwaldow I updated the update description. Feel free to review it and give feedback

@vineetasharma105
Copy link

vineetasharma105 commented Nov 29, 2024

Issue in the preview of the layers harvested from remote service.
-As a sample, this dataset from external service on the preview shows the Holisoils banner as can be seen here :
image
This preview issue exists only for layers from remote services.
-For 'external applications' , the preview show 'Bookmark' issue :
image
-The previews in case of other layers are working fine.

@vineetasharma105
Copy link

Also, the preview on geonode-demo is working fine as can be seen here :
image
But the same layer when made subsite exclusive, shows the erroneous preview :
image

@ridoo
Copy link
Collaborator

ridoo commented Dec 3, 2024

@vineetasharma105

  • The given example dataset is not public so I cannot have a look unfortunately The other examples you gave are accessible from the subsite -- I will give them a quick test. My current guess is, that you observe a side effect from the slug changes Mattia did to support exclusive datasets for subsites. Perhaps remote services are not supported yet (all other public dataset seem to work fine).
  • For externalapplications the subsites URL slug configuration would have to be extended to included the externalapplications

@vineetasharma105
Copy link

Hi Henning,
Yes , I also not exclusively had a look at the access options for anonymous users as the default 'allowed permissions' are view and download. Still I now made those layers 'Viewable' for anyone via the frontend.
Please let me know if in case any other parts have to be taken care.

@ridoo
Copy link
Collaborator

ridoo commented Dec 3, 2024

There are recent changes made by Mattia regarding remote services. However, locally I make the same observation you did. Perhaps, I have to adjust integrate his _geonode_config.html configuration (which actually should be automatically respected .. I will check)

@ridoo
Copy link
Collaborator

ridoo commented Dec 3, 2024

@vineetasharma105 I fixed the externalapplication issue. It now shows the thumbnail instead of trying to render an embedded view:

image

Also the remote service instance should be ok now as well:

image

@ridoo
Copy link
Collaborator

ridoo commented Dec 3, 2024

@gannebamm @vineetasharma105 @hvwaldow I updated the thuenen_atlas which includes the latest fixes.

@gannebamm
Copy link
Contributor Author

Is deployed

@vineetasharma105
Copy link

Henning, it could also be that because geonode-demo is not upgraded yet , I am not able to add/create external application in subsite. For eg. I am here not getting the option in the 'Add resource' list
image
Although, if i manage to filter resources with 'external resources', then I not only get the filtered external resources, but now also getting the option to add a resource as external application :
image
But again, the newly created external application resource will be part of geonode-demo but not the holisoils subsite and the process of adding the group 'holisoils' and subsite_exclusive keyword will shift this resource to subsite contents. The preview is still problematic. Let me check it on the geonode-demo with the latest fixes you mentioned.

@ridoo
Copy link
Collaborator

ridoo commented Dec 3, 2024

@vineetasharma105 that externalapplications are accessible only from catalogue is a known issue. For now, it is a known limitation. I will have to check, if we have resources left to dig deeper here.

But again, the newly created external application resource will be part of geonode-demo but not the holisoils subsite and the process of adding the group 'holisoils' and subsite_exclusive keyword will shift this resource to subsite contents.

That is similar to remote services, as the integration of adding externalapplications are still based on Django forms.

Please check the preview issue. It worked for me locally.

@gannebamm
Copy link
Contributor Author

@ridoo Sorry, I messed up (again) with the rebuild and old versions of the submodules were used. Vineeta is doing a recheck at the time of this writing.

@vineetasharma105
Copy link

vineetasharma105 commented Dec 4, 2024

Hi @ridoo :
Florian just told me about how to make the external application public and I apologize I did not know this before. All I wanted to do was to make it public so that you can also view it and I can present the resource where the preview issue was there on the geonode-demo/holisoils.
But I have checked it now and the preview is working fine :
image
👍

@ridoo
Copy link
Collaborator

ridoo commented Dec 4, 2024

@vineetasharma105 @gannebamm I took some time now to investigate what is wrong with the non-paged route / and why externalapplications menu items are not visible (but visible on other, paged routes).

It seems to be an issue on upstream geonode-mapstore-client where overridden config does not get applied for / for some reason. I created an upstream issue and documented it on the externalapplication contrib module repository (known issue).

The only workaround I see is to add the externalapplication menu entry as code in our geonode-mapstore-client fork. I am about to apply the workaround on Thuenen-GeoNode-Development/geonode-mapstore-client and will provide an updated thuenen_atlas once everything is in place.

Edit: External Application is now available from the "Add ResourceMenu even on/ `.

@gannebamm you can re-deploy the latest changes
@vineetasharma105 feel free to test .. however, the subsite_exclusive and group settings still have to be done via Django Admin (as for remote services)

Edit: More todo 😅 : https://github.com/Thuenen-GeoNode-Development/Sprints/issues/10#issuecomment-2517163495

@ridoo
Copy link
Collaborator

ridoo commented Dec 4, 2024

I addressed the TODO mentioned in https://github.com/Thuenen-GeoNode-Development/Sprints/issues/10#issuecomment-2517163495. When successfully tested, this should mark the final task for this issue.

@vineetasharma105
Copy link

Hi Henning,
The 'external applications' option is now available when adding a resource on the first page. And also working.

@ridoo
Copy link
Collaborator

ridoo commented Dec 11, 2024

@gannebamm @vineetasharma105 can this be closed then?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation EPIC Maintenance LB 2.1.3 question Further information is requested
Projects
None yet
Development

No branches or pull requests

3 participants