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

Merge 0.4.1 from testing to master #63

Merged
merged 9 commits into from
Nov 2, 2022
Merged

Merge 0.4.1 from testing to master #63

merged 9 commits into from
Nov 2, 2022

Conversation

MayeulC
Copy link
Collaborator

@MayeulC MayeulC commented Oct 27, 2022

Problem

  • Outdated master branch

Solution

  • Merge testing

PR Status

  • Code finished and ready to be reviewed/tested
  • (WIP) 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)

@MayeulC
Copy link
Collaborator Author

MayeulC commented Oct 27, 2022

!testme

@yunohost-bot
Copy link
Contributor

🌻
Test Badge

@MayeulC
Copy link
Collaborator Author

MayeulC commented Oct 27, 2022

Weird CI error, looks like synapse was not installed, I relaunched it.

Confirmed working on my system, but it only worked after I installed an updated signald, and restarted both mautrix_signal and signald: #22 (comment)

@nathanael-h
Copy link
Contributor

Confirm upgrade from master (0.3.0) works on my server. I did not had the signald upgrade issue.

* Upgrade to v0.4.1

* Auto-update README

* Update app.src (#66)

change facebook to signal

Co-authored-by: yunohost-bot <[email protected]>
Co-authored-by: yunohost-bot <[email protected]>
Co-authored-by: Thatoo <[email protected]>
@Gredin67 Gredin67 changed the title Merge 0.4 from testing to master Merge 0.4.1 from testing to master Nov 2, 2022
@Gredin67
Copy link
Collaborator

Gredin67 commented Nov 2, 2022

!testme
Should be merged ASAP because all bridges are broken #64 because of https://gitlab.com/signald/signald/-/issues/343

@yunohost-bot
Copy link
Contributor

Meow 🐈
Test Badge

@yunohost-bot
Copy link
Contributor

✌️
Test Badge

@MayeulC MayeulC merged commit 350f877 into master Nov 2, 2022
@MayeulC
Copy link
Collaborator Author

MayeulC commented Nov 2, 2022

Merged as suggested by @Gredin67 :)

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