Skip to content

Conversation

daniel-jettka
Copy link
Contributor

@daniel-jettka daniel-jettka commented Sep 24, 2025

Description, Context and related Issue

Frontend host url was not correctly derived. Changed this.
Now the about window shows correct info fetched from backend and frontend's CITATION.cff

Refs #106

image

How Has This Been Tested?

Tested with EditionExample and frontend running on nginx http://localhost:8089 and in eXist-db http://localhost:8080/exist/apps/Edirom-Online-Frontend/

Docker compose with respective frontend branch and edition xar specified (and unset afterwards):
export FE_BRANCH=fix-106-about-window-not-displaying-content && export EDITION_XAR=https://github.com/Edirom/EditionExample/releases/download/v0.1.1/EditionExample-0.1.1.xar && docker compose down --volumes --remove-orphans && docker compose build --no-cache && docker compose up -d && unset FE_BRANCH && unset EDITION_XAR

Types of changes

  • Bug fix (non-breaking change which fixes an issue)

Overview

  • I have updated the inline documentation accordingly.
  • I have performed a self-review of my code, according to the style guide
  • I have read the CONTRIBUTING document.

Copy link
Contributor

@hizclick hizclick left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Works on Chrome, Firefox, and Safari, with no error.
Thank you

@github-project-automation github-project-automation bot moved this from In Progress to Approved in Edirom Development Sep 24, 2025
@daniel-jettka daniel-jettka merged commit 7de2497 into develop Sep 24, 2025
1 check passed
@daniel-jettka daniel-jettka deleted the fix-106-about-window-not-displaying-content branch September 24, 2025 13:45
@github-project-automation github-project-automation bot moved this from Approved to Done in Edirom Development Sep 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

[BUG] About window not displaying content

2 participants