administration: adds harvester view#701
administration: adds harvester view#701jrcastro2 wants to merge 2 commits intoCERNDocumentServer:masterfrom
Conversation
jrcastro2
force-pushed
the
add-harvest-reports
branch
from
ddef5d2 to
ee55b93
Compare
| /** | ||
| * Custom Empty Results component (without showing the query) | ||
| */ | ||
| export const CustomEmptyResults = () => { |
There was a problem hiding this comment.
Added this to hide the query from the user, the default one shows it and we are "manipulating" it ot add the timestamp
There was a problem hiding this comment.
I have a feeling this could have been solved by hidden_params in search view
| const { sortOptions, sortOrderDisabled } = useContext(SearchConfigurationContext); | ||
|
|
||
| // Derive selected run from query string | ||
| const runIdFromQuery = extractRunIdFromQuery(currentQueryState.queryString, runs); |
There was a problem hiding this comment.
Couldnt find a better way. Happy to hear other ideas but I had issues with RSK storing the value of the run in this cmp. It would always fall back to initalquery set and not properly store it. So cant keep track of which run is being showed (unless the run id would be part of the URL params, since the audit logs have no reference to it, it doesn't make any sense to me)
There was a problem hiding this comment.
let's chat in a call about this... I need more context to understand what is happening.
I mostly understand that this does some kind of pre-filtering? Wouldn't using hidden_params solve it?
| /** | ||
| * Extract user's search terms from full query string | ||
| */ | ||
| export const extractUserSearch = (queryString) => { |
There was a problem hiding this comment.
Since we need to "enrich" the URL with the timestamp information to select a run we want to hide this from the user
jrcastro2
force-pushed
the
add-harvest-reports
branch
2 times, most recently
from
943fd8e to
f23971e
Compare
| from cds_rdm.permissions import can_access_administration_menu | ||
|
|
||
|
|
||
| def require_admin_or_harvester_curator(f): |
There was a problem hiding this comment.
see my comment in the invenio-administration PR
| ] | ||
|
|
||
| @staticmethod | ||
| def disabled(): |
There was a problem hiding this comment.
IMO No need to hide this behind a feature flag
| for run in runs | ||
| ] | ||
|
|
||
| def get_context(self, **kwargs): |
There was a problem hiding this comment.
this was actually more than what I had in mind, nice UX addition, thanks!
| return; | ||
| } | ||
|
|
||
| const downloadUrl = `/harvester-reports/download?q=${encodeURIComponent(query)}`; |
There was a problem hiding this comment.
I ment this endpoint more to be included in the email, but this works as well
jrcastro2
force-pushed
the
add-harvest-reports
branch
from
f23971e to
a778af4
Compare
* Restrict harvester-curator to get only system user actions
jrcastro2
force-pushed
the
add-harvest-reports
branch
from
a778af4 to
5c3bce1
Compare
2 participants
Showcase
Grabacion.de.pantalla.2026-02-18.a.las.13.46.11.mov
Email example