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

Upgrade #72

Merged
merged 46 commits into from
Aug 15, 2022
Merged

Upgrade #72

merged 46 commits into from
Aug 15, 2022

Conversation

Gredin67
Copy link
Collaborator

@Gredin67 Gredin67 commented Dec 15, 2021

Problem

  • Level 0

Solution

  • Fix install

PR Status

  • Code finished and ready to be reviewed/tested
  • The fix/enhancement were manually tested (if applicable)

Automatic tests

Automatic tests can be triggered on https://ci-apps-dev.yunohost.org/ after creating the PR, by commenting "!testme", "!gogogadgetoci" or "By the power of systemd, I invoke The Great App CI to test this Pull Request!". (N.B. : for this to work you need to be a member of the Yunohost-Apps organization)

@zamentur
Copy link
Contributor

!testme

@yunohost-bot
Copy link
Contributor

Fingers crossed!
Test Badge

@zamentur
Copy link
Contributor

zamentur commented Mar 18, 2022

TODO:

  • Fix #75
  • Fix permission issues
  • Fix configuration upgrade

@zamentur zamentur changed the title Upgrade 6.4.2 Testing Mar 31, 2022
@zamentur zamentur marked this pull request as ready for review March 31, 2022 14:59
@ericgaspar
Copy link
Member

!testme

@ericgaspar
Copy link
Member

!testme

@yunohost-bot
Copy link
Contributor

✌️
Test Badge

@zamentur
Copy link
Contributor

ttf-mscorefonts-installer is now required by onlyoffice, so contrib repo should be enabled.

@ericgaspar
Copy link
Member

I just thought i would add ttf-mscorefonts-installer to extra_dependencies do we need to do something else?

@zamentur
Copy link
Contributor

Yes cause this package is in contrib debian repo, not just in main component. So i had to update /etc/apt/sources.list

In more i had to extract the default.json cause it's not updated by our helpers:

dpkg-deb --fsys-tarfile /var/cache/apt/archives/onlyoffice-documentserver_7.1.0-215_amd64.deb | tar -x ./etc/onlyoffice/documentserver/default.json

@yalh76
Copy link
Member

yalh76 commented Jul 13, 2022

!testme

@yunohost-bot
Copy link
Contributor

Fingers crossed!
Test Badge

@yalh76 yalh76 requested review from ericgaspar and zamentur July 14, 2022 13:06
@yalh76 yalh76 changed the title Testing Upgrade Jul 14, 2022
@yalh76
Copy link
Member

yalh76 commented Jul 20, 2022

!testme

@yunohost-bot
Copy link
Contributor

🌻
Test Badge

@yalh76
Copy link
Member

yalh76 commented Jul 20, 2022

@zamentur : do you think it's ok to merge ?

@yalh76 yalh76 merged commit 2be500c into master Aug 15, 2022
rodinux added a commit to rodinux/onlyoffice_ynh that referenced this pull request Apr 23, 2023
it can not works because too many changes.
rodinux added a commit to rodinux/onlyoffice_ynh that referenced this pull request Apr 23, 2023
too many changes with the new release
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.

5 participants