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

geonetwork - update submodule #2871

Merged
merged 1 commit into from
Jan 7, 2020
Merged

geonetwork - update submodule #2871

merged 1 commit into from
Jan 7, 2020

Conversation

pmauduit
Copy link
Member

@pmauduit pmauduit commented Jan 7, 2020

Bump to version 3.8.2 (see #2431)

Tests: as far as I remember, utests & itests ok, some runtime tests (locally and onto dev.geograndest.fr/geonetwork)

Note: there are still at least one known issue about this version: the default configuration of the UI is broken and requires some fiddling around to be fixed (you will see 2 "full view" tab entry in the MD view), ping @cmangeat

@pmauduit pmauduit added this to the 19.12 milestone Jan 7, 2020
@pmauduit pmauduit added this to To do in DataGrandEst via automation Jan 7, 2020
@fvanderbiest fvanderbiest merged commit c4c6c1e into master Jan 7, 2020
DataGrandEst automation moved this from To do to Done Jan 7, 2020
@fvanderbiest fvanderbiest deleted the geonetwork-3.8 branch January 7, 2020 11:46
@landryb
Copy link
Member

landryb commented Jan 7, 2020

Note: there are still at least one known issue about this version: the default configuration of the UI is broken and requires some fiddling around to be fixed (you will see 2 "full view" tab entry in the MD view), ping @cmangeat

that's what i'd call a nice blocker :) is there a tracking bug for it ? isnt it just a matter of yet again updating the default schema_plugins in the geonetwork default datadir ?

@fxprunayre
Copy link
Member

the default configuration of the UI is broken

You should probably create a proper default UI config for geOrchestra in https://github.com/geonetwork/core-geonetwork/blob/master/web-ui/src/main/resources/catalog/js/CatController.js#L50-L285 as you don't want to use the default GN one (eg. use external viewer) and with that, at least all users can easily reset to the default from the admin:

image

and there is still room for improvements for updating this config during version updates combining existing with updates ...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
DataGrandEst
  
Done
Development

Successfully merging this pull request may close these issues.

None yet

4 participants