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

Copy custom config files to docker image #208

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

hakonhagland
Copy link
Collaborator

Copies a snapshot of LibreOffice config files into the docker image. The config files have been modified from the default LibreOffice config files in two ways:

Copys libreoffice config files into docker image. The config files have
been modifed from the default libreoffice config files in two ways:
First, document comparison mode has been disabled, see PR OPM#204 for
details. Second, the parts sub directory inside the image has been added
as a trusted location to get rid of the popup dialog "Would you like to
change the document, and update all links to get the most recent data?"
@blattms
Copy link
Member

blattms commented Apr 12, 2024

First, document comparison mode has been disabled, see PR #204 for details,

Will this still allow us to compare two different documents (Open the update one, Edit -> Track changes -> Compare Document, and use the old version)? That option is actually quite handy.

@hakonhagland
Copy link
Collaborator Author

@blattms I am not sure why we we would use libreoffice to compare two different documents when we already have git version control? Can you elaborate on that idea?

Anyway, as I remember it, this pull request was in accordance with the README.md file which suggests that we turn off document comparison mode. If the user still want document comparison he could he could avoid using the docker container version of libreoffice or he could manually re-enable the mode from the menu in libreoffice. But it is not recommended if he plans to submit a clean PR..

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.

None yet

2 participants