Skip to content

Conversation

@milanmajchrak
Copy link
Collaborator

Phases MP MM MB MR JM Total
ETA 0 0 0 0 0 0
Developing 0 0 0 0 0 0
Review 0 0 0 0 0 0
Total - - - - - 0
ETA est. 0
ETA cust. - - - - - 0

Problem description

milanmajchrak and others added 30 commits September 20, 2024 08:20
* add tar for preview
* The type-bind is loaded from the cfg and correctly rendered

* Done some refactoring

* Update tests following the type-bind enhancement

* Fixed linting error.

* Removed unwanted changes and fixed wrong refactored code.
* Added static files from the SEZNAM license.

* Refactored fetching html content from the static files. The logic was moved into the common service.

* Show Seznam static license on approval page.

* Updated constant name LICENSE_NAME_SEZNAM - added the _CZ to make it more clear that is Czech license
* Added share submission button into workspaceitem actions page.

* Added notification about the success of the sharing the submission.

* Created share submission module, page with routing. That page could access only the administrator.

* WIP - created a new page `change-submitter-page`.

* Created page when the user could take the workspace item as its own.

* Pretified the code and added some docs

* Updated tests following the new feature

* Fixed unwanted changes, updated docs
* Added support for searching results from specific solr indexes.

Updated autocomplete component to search values from custom solr index. The autocomplete component could have attribute for resolving the custom index from the submission-forms definition.

* Show suggestions for `solr-handle_title_ac` and `solr-subject_ac`

* Fixed tests

* Formatted language suggestions, added docs and fixed tests.

* Refactored the code following the review requirements.

* Fixed type in the word pretify
* Removed flex class and used width style instead of col

* Removed unsued class

* Changed style in the html into classes
* Migrated Vanilla cs messages into clarin 7.6.1. cs messages

* updated translation

* Updated cs localization for subcommunities and subcollections

* Changed `prispeveku` to `prispevku`

* Updated messages for the 'supervised' and 'claim' sentenses

---------

Co-authored-by: Ondrej Kosarko <[email protected]>
* The submitter information cannot be displayed in some cases, but show a few information about the Item

* Added missing trailing comma

* Fixed docs
1. Author does not have a two input fields.
2. Test for checking the notice step will be transformed into UI test, because we do not want to keep such specific collection for generic IT.
* added licenses from ufal commit dcbe87e02931dee78b13c4b3995a6cef3aeacd32

* added missing static licenses

* remove static from licenses path

* no_static_ as const

* checkstyle
* Use NAMESPACES from the .env

* Created symbolic link for the tomcat folder

* Updated docker compose files - can load more properties from the `.env`
Because some tests were canceled during processing.
Uncommented /repository/ namespace in the docker-compose.yml.
It is only for one deploy.
Undo exposing server on the /repository/ path
…se in the container it is always `/` and `/server` (#741)
…ken (#733)

* Used hardredirect instead of the standard router.redirect because the user could be stucked on the loading page

* Added doc why

* Added missing whitespace
milanmajchrak and others added 28 commits May 14, 2025 13:39
* Do not use the clarin item view box for the bulk access

* Removed unused import

* Clarify the `showClarinViewBox` is boolean

* Use the constant for the hardcoded bulk access list id
* Create Acknowledgment-ReadMe.md

Acnkowledgment of NRP project

(cherry picked from commit ad889b2)

* Video files previews

This uses the thumbnail as poster (if available) and correctly sets the
source of the video

currently only works for anonymously accessible files.

(cherry picked from commit 4832c2f)

* Handle video previews for restricted items

append a shortlived token at the right time (error, seeking, stalled)

(cherry picked from commit 2c12d7d)

* Display only ORIGINAL bitstreams

Thumbnails, when available, should be shown istead of the generic
MIME_TYPE_IMAGE.

Content of the TEXT bundle should not be shown at all this is usually
automatically extracted "text layer" of a PDF, useful for indexing, but
don't want people downloading it.

(cherry picked from commit 3862442)

* fix linter and test errors

(cherry picked from commit f852096)

* Code review follow up

the listOfFiles should really not contain files from "TEXT" or
"THUMBNAIL" bundles.

* code review

unsubscribe error$, seeking$ and $stalled

* code review - thumbnail might be undefined

* code review - consistent formatting
* Design fix - added spaces between links

* Fix of firstMedataValue null properties
…oad after agreeing (#874)

* Changed logic (condition) of retrieving file download link

* Saved the original logic, but changed caching parameter
…ed in the URL, because that query part of the URL was encoded as the filename. (#873)
* Playright after deploy running here

* Using different token

* Corrected path in copying json file

* Checkout only branch in UI tests not master

* Added copilot changes

---------

Co-authored-by: Matus Kasak <[email protected]>
* use `$DSPACE_REST_NAMESPACE` for the webapps symlink

(cherry picked from commit 9ea936e)

* Licenses sync

(cherry picked from commit 4ee2f1c)

* update header/footer

based on latest lindat-common 3.5.0

(cherry picked from commit 7b2f299)

* expose both tcp & udp ports of the handle server

(cherry picked from commit 46bc936)

* Update docker-compose-rest.yml

replace ../webapps with /usr/local/tomcat/webapps

---------

Co-authored-by: Ondrej Kosarko <[email protected]>
Co-authored-by: Ondrej Kosarko <[email protected]>
…work properly (#881)

* hotfix - finding index of unchecked box properly to formName arg
* locked searchbar when searchOption is not picked

* edited Resource Type search option to not be case-sensitive

* edited Resource Type search option to show nothing when there is no match

* try to set lowercase constants without toLowerCase method

* Added full list of handles for empty Resource Type search option

* refactored empty searchQueries handling and added comment for invalid resourcetype search inputs
…cted (#897)

* Show openaire input field every time the EU type is selected
* copied static content

that was not already in cs/.

Otherwise license page (when without .html) might show an error when UI
is in Czech and there's no translation.

(cherry picked from commit 9e7eea0)

* Update header.component.html

I'm expecting the image to take me to lindat.cz. There's no other obvious way to navigate out of the repository

* This is the change from ufal/clarin-dspace@5fc2414

but applied on v7

* Adding a note about the proprietor change

---------

Co-authored-by: Ondrej Kosarko <[email protected]>
Co-authored-by: Michal Kren <[email protected]>
* show msg about no files when item has no files, show no spinenr

* check if files are array

* check files size

* fix error

* optimize template with hasNoFiles flag

* used behaviourSubject for hasNoFiles / aynchronous
* Created trigger for running UI tests for each customer daily

* Corrected spaces

* Changed name of the tests

---------

Co-authored-by: Matus Kasak <[email protected]>
* Added loading spinner icon after 'I agree' button is clicked

* Removed type annotation and rephrased comment

* Changes isLoading to async object

* Edited loading spinner visibility method - consistency fix
* The user always see it's submissions page

---------

Co-authored-by: Paurikova2 <[email protected]>
…but 403 (#920)

* Updated retrieving the dso object - redirect to 403 instead of login page. Check if the user is signed in.

* Updated commend and fixed eslint warning
* Added `authorization-token` to the request, so the user could be identified in the BE context

* Added notification error when downloading has failed
were missing an update in the czech translation.

Co-authored-by: Ondrej Kosarko <[email protected]>
#930)

* Fetch the refbox content from the BE

* Show an error that the refbox content wasn't properly fetched when some error

* Sanitize the code from the BE. Show errors and proper messages when something has failed

* Load item name in on init

* Secure the refbox content using SafeHtml

* Make refbox variable name consistent

* Refactor fetching refbox string from the SafeHtml
@coderabbitai
Copy link

coderabbitai bot commented Jul 24, 2025

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

✨ Finishing Touches
🧪 Generate unit tests
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch lindat-merge-dtq-dev-2025-07-24

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share
🪧 Tips

Chat

There are 3 ways to chat with CodeRabbit:

  • Review comments: Directly reply to a review comment made by CodeRabbit. Example:
    • I pushed a fix in commit <commit_id>, please review it.
    • Explain this complex logic.
    • Open a follow-up GitHub issue for this discussion.
  • Files and specific lines of code (under the "Files changed" tab): Tag @coderabbitai in a new review comment at the desired location with your query. Examples:
    • @coderabbitai explain this code block.
    • @coderabbitai modularize this function.
  • PR comments: Tag @coderabbitai in a new PR comment to ask questions about the PR branch. For the best results, please provide a very specific query, as very limited context is provided in this mode. Examples:
    • @coderabbitai gather interesting stats about this repository and render them as a table. Additionally, render a pie chart showing the language distribution in the codebase.
    • @coderabbitai read src/utils.ts and explain its main purpose.
    • @coderabbitai read the files in the src/scheduler package and generate a class diagram using mermaid and a README in the markdown format.
    • @coderabbitai help me debug CodeRabbit configuration file.

Support

Need help? Create a ticket on our support page for assistance with any issues or questions.

Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments.

CodeRabbit Commands (Invoked using PR comments)

  • @coderabbitai pause to pause the reviews on a PR.
  • @coderabbitai resume to resume the paused reviews.
  • @coderabbitai review to trigger an incremental review. This is useful when automatic reviews are disabled for the repository.
  • @coderabbitai full review to do a full review from scratch and review all the files again.
  • @coderabbitai summary to regenerate the summary of the PR.
  • @coderabbitai generate docstrings to generate docstrings for this PR.
  • @coderabbitai generate sequence diagram to generate a sequence diagram of the changes in this PR.
  • @coderabbitai generate unit tests to generate unit tests for this PR.
  • @coderabbitai resolve resolve all the CodeRabbit review comments.
  • @coderabbitai configuration to show the current CodeRabbit configuration for the repository.
  • @coderabbitai help to get help.

Other keywords and placeholders

  • Add @coderabbitai ignore anywhere in the PR description to prevent this PR from being reviewed.
  • Add @coderabbitai summary to generate the high-level summary at a specific location in the PR description.
  • Add @coderabbitai anywhere in the PR title to generate the title automatically.

CodeRabbit Configuration File (.coderabbit.yaml)

  • You can programmatically configure CodeRabbit by adding a .coderabbit.yaml file to the root of your repository.
  • Please see the configuration documentation for more information.
  • If your editor has YAML language server enabled, you can add the path at the top of this file to enable auto-completion and validation: # yaml-language-server: $schema=https://coderabbit.ai/integrations/schema.v2.json

Documentation and Community

  • Visit our Documentation for detailed information on how to use CodeRabbit.
  • Join our Discord Community to get help, request features, and share feedback.
  • Follow us on X/Twitter for updates and announcements.

@milanmajchrak milanmajchrak merged commit 6003cbd into customer/lindat Jul 24, 2025
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants