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

VREM Upgrade #5

Merged
merged 53 commits into from
Jul 23, 2021
Merged

Conversation

simonpeterhans
Copy link
Member

@simonpeterhans simonpeterhans commented Jul 22, 2021

Front end rework PR: VIRTUE-DBIS/virtual-exhibition-presenter#11

This will break the UI; I have a (cheap) fix but won't find the time to finalize it until I'm back to work in ~2 weeks.

The example exhibition at https://github.com/VIRTUE-DBIS/vre-mixnhack19 could probably also use a rework (I can also do that as soon as I'm back).

Major changes:

  • Migrate to Kotlin 1.5 (and Java 11)
  • Clean up build script and dependencies (KMongo for MongoDB, Kotlinx for Serialization, Javalin as web fw, kotlin-logging for logging)
  • Serving is no longer limited to a single exhibition; instead, any (imported) exhibition can be requested at runtime (importing exhibitions stores their images in data/ relative to the pwd so they can be served upon being requested)
  • A lot of refactoring and tidying
  • Cineast integration (already included as discussed with @silvanheller):
    • Cineast API code is generated (via generateOpenApi) and ready to be used; Cineast settings can be configured in the config file (but don't have to if Cineast is not used to serve content)
    • Content from Cineast can be served by ending a path via .remote (this will be used for generated collections later on)

sauterl and others added 30 commits January 18, 2020 23:55
@silvanheller
Copy link
Member

I'll merge this, and we'll do a v0.2 release once we're confident that this is stable (e.g. we've had another person do a full test including setup)

@silvanheller silvanheller merged commit 231a8bc into VIRTUE-DBIS:master Jul 23, 2021
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.

4 participants