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

Moving Data Distribution API into the Vitro codebase #484

Open
wants to merge 9 commits into
base: main
Choose a base branch
from

Conversation

litvinovg
Copy link
Contributor

VIVO GitHub issue

  • Other Relevant Links (Mailing list discussion, related pull requests, etc.)

What does this pull request do?

Added the Data Distribution API and report generation into Vitro codebase.

Additional Notes:

Any additional information that you think would be helpful when reviewing this PR.

Interested parties

@chenejac @hauschke @brianjlowe

Reviewers' expertise

Candidates for reviewing this PR should have some of the following expertises:

  1. Java
  2. HTML, CSS, JavaScript
  3. FreeMarker
  4. SPARQL
  5. Ontologies
  6. Natural language knowledge
    1. English
    2. German
    3. Spanish
    4. French
    5. Portuguese
    6. Russian
    7. Serbian

Reviewers' report template

Please update the following template which should be used by reviewers.

General comment

A reviewer should provide here comments and suggestions for requested changes if any.

Testing

A reviewer should briefly describe here how it was tested

Code reviewing

A reviewer should briefly describe here which part was code reviewed

Sandra Mierz and others added 9 commits January 16, 2025 09:52
takeover from graham

rm unimplemented or unnecessary classes

rm unused distributors

added us text for vqt

use local server port to make new request instead of port used by the client

fix self signed certificate issue

Renamed german translation properties

checkstyle fixes
This reverts commit 7c2f665991b94c8d9f7e4f38342502815a4be8f7.
This reverts commit 915f96dc8549f11aad44c62a21f5096e02345528.
@litvinovg litvinovg requested a review from chenejac January 16, 2025 09:57
@hauschke
Copy link
Member

@markuskotte agreed to review German language parts, when and if necessary.

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.

2 participants