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

Fix an issue with Ubuntu Server 23.04 where download form App-store i… #271

Merged
merged 2 commits into from
May 19, 2023

Conversation

marioqxx
Copy link
Contributor

…ssues an error when the app is already installed, but disabled. Also no download is then necessary.

…ssues an error when the app is already installed, but disabled. Also no download is then necessary.

Signed-off-by: marioqxx <marioqxx@users.noreply.github.com>
@marioqxx marioqxx force-pushed the main branch 3 times, most recently from 21cb708 to 0332968 Compare May 14, 2023 13:05
@marioqxx
Copy link
Contributor Author

marioqxx commented May 14, 2023

I do have another pull-request in my queue and wanted to add this further pull-request. This somehow messed everything up and combined the pull-requests - whcih concern very different subjects and shall not be mixed. Also because the next pull-request proposal has significant more changes. I still struggle with git and github, but its getting better ... . I seem to have this rolled back and all is where it should be.

@wiktor2200 wiktor2200 added the bug Something isn't working label May 19, 2023
@wiktor2200
Copy link
Collaborator

Hi @marioqxx! Sorry for late response. Approved, code looks good. Looking forward for your next PR.
BTW, the easiest way to work with multiple forks and branches is to checkout from main. It could give you some conflicts but should be relatively easy to solve (automatically in most cases) :)

@wiktor2200 wiktor2200 merged commit efa00b0 into nextcloud:main May 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants