Skip to content

Conversation

@fxprunayre
Copy link
Member

This is mainly to improve search statistics reports and have a more representative search service information.

image

Currently "qi" which could be defined as internal search service is never used by the Angular app - as it was in the ExtJS app.

… app.

This is mainly to improve search statistics reports.
@fxprunayre fxprunayre added this to the 3.4.2 milestone Feb 7, 2018
@Delawen
Copy link
Contributor

Delawen commented Feb 8, 2018

This adds an "internal" param, but I don't see anything on the server side. Are you going to add some UI on the statistics?

@fxprunayre
Copy link
Member Author

This adds an "internal" param, but I don't see anything on the server side

There is 2 search services: q and qi which are the same but allows to make distinction between main search form used by users and application searches required for the app (eg. home page stats, last records, list of records related to a directory entry). The old Ext apps was using properly those 2 services but the angular app always use the q service. So this makes the stats useless because most of the searches are made by the app and we can't find the user search in current search stats.

Are you going to add some UI on the statistics?

This was done in #2023

@Delawen Delawen requested review from Delawen and jahow February 8, 2018 12:30
@fxprunayre fxprunayre merged commit f7e3267 into geonetwork:3.4.x Mar 14, 2018
@fxprunayre fxprunayre self-assigned this Mar 14, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants