-
-
Notifications
You must be signed in to change notification settings - Fork 79
WIP F-Droid market #698
base: develop
Are you sure you want to change the base?
WIP F-Droid market #698
Conversation
DIOHz0r
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
CI is failing and you didn't complete pomodoro info
|
@DIOHz0r |
178121a to
7e44ddc
Compare
|
There were the following issues with this Pull Request
You may need to change the commit messages to comply with the repository contributing guidelines. 🤖 This comment was generated by commitlint[bot]. Please report issues here. Happy coding! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There are accessibility issues in these changes.
| $png = 'iVBORw0KGgoAAAANSUhEUgAAAAEAAAABCAYAAAAfFcSJAAAADUlEQVR42mNkYPhfDwAChwGA60e6kgAAAABJRU5ErkJggg=='; | ||
|
|
||
| $instance = $this->newTestedInstance(); | ||
| $output = $instance::getSpecificValueToDisplay('icon', ['icon' => $png]); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You're missing an alt attribute here. This is a problem for people using screen readers. Here are some tips for what to write.
|
There were the following issues with this Pull Request
You may need to change the commit messages to comply with the repository contributing guidelines. 🤖 This comment was generated by commitlint[bot]. Please report issues here. Happy coding! |
|
I solved all merge conflicts which will occur with release/2.0.0, except one. We must wait the merge of the release branch, then rebase this one, solve the conflict, then merge into develop. Checking the build remains greek for now. |
|
There were the following issues with this Pull Request
You may need to change the commit messages to comply with the repository contributing guidelines. 🤖 This comment was generated by commitlint[bot]. Please report issues here. Happy coding! |
|
There were the following issues with this Pull Request
You may need to change the commit messages to comply with the repository contributing guidelines. 🤖 This comment was generated by commitlint[bot]. Please report issues here. Happy coding! |
|
There were the following issues with this Pull Request
You may need to change the commit messages to comply with the repository contributing guidelines. 🤖 This comment was generated by commitlint[bot]. Please report issues here. Happy coding! |
e01ee68 to
794e277
Compare
Signed-off-by: Thierry Bugier <[email protected]>
Signed-off-by: Thierry Bugier <[email protected]>
Signed-off-by: Thierry Bugier <[email protected]>
Signed-off-by: Thierry Bugier <[email protected]>
Signed-off-by: Thierry Bugier <[email protected]>
Signed-off-by: Domingo Oropeza <[email protected]>
Signed-off-by: Domingo Oropeza <[email protected]>
Signed-off-by: Domingo Oropeza <[email protected]>
Signed-off-by: Domingo Oropeza <[email protected]>
c724440 to
4cf0db4
Compare
|
Hello, I did a rebase here for fixing the conflicts. |
|
I still need to validate that updates of apps in fdroid propagate automatically to devices when needed |
…nto feature/fdroid_2
rebase of #193
provides a way to get apps from fdroid, download them as new packages and update them automatically.
Changes description
Checklist
Please check if your PR fulfills the following specifications:
Estimated time
References
Closes #N/A
Related #N/A
Depends on #N/A