Skip to content

Update dependencies, upgrade preferences and goodbye to anko#55

Merged
raphaelm merged 7 commits into
masterfrom
upgrade-bye-anko
Mar 17, 2023
Merged

Update dependencies, upgrade preferences and goodbye to anko#55
raphaelm merged 7 commits into
masterfrom
upgrade-bye-anko

Conversation

@robbi5

@robbi5 robbi5 commented Mar 6, 2023

Copy link
Copy Markdown
Contributor

(Currently based on the pin-dialog #54 branch, will be rebased when that got merged)

This PR updates gradle, kotlin and some dependencies, introduces flipper in pretixPRINT too, migrates our settings/preferences to androidx, the styling to Material3 and finally completely removes anko \o/
I've tried to split the parts into single commits for easier review.

TODO:

  • Update about.html with dependency licenses

@robbi5 robbi5 requested a review from raphaelm March 6, 2023 18:20
@robbi5 robbi5 changed the title Update dependencies and goodbye to anko Update dependencies, upgrade preferences and goodbye to anko Mar 6, 2023

@raphaelm raphaelm left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

#54 is merged, so you can finish this one up at your convenience :) On a quick glance, I could not find obvious problems with this PR.

@robbi5 robbi5 force-pushed the upgrade-bye-anko branch from 7eecca8 to 627763c Compare March 16, 2023 15:28
<?xml version="1.0" encoding="utf-8"?>
<manifest xmlns:android="http://schemas.android.com/apk/res/android"
xmlns:tools="http://schemas.android.com/tools"
package="eu.pretix.pretixprint">

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is that removal on purpose?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yep, Android Studio removed that automatically, it is now replaced by the applicationId in build.gradle

@raphaelm raphaelm merged commit 9be6d29 into master Mar 17, 2023
@raphaelm raphaelm deleted the upgrade-bye-anko branch March 17, 2023 08:56
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.

2 participants