Skip to content
This repository has been archived by the owner on Oct 17, 2021. It is now read-only.

2021-05-06 v. 1.0.7: added in-app update feature #52

Merged
merged 3 commits into from May 7, 2021

Conversation

vipozdn
Copy link
Contributor

@vipozdn vipozdn commented May 6, 2021

1.Added:

  • play core library dependency in app/build.gradle
  • flexible in-app update feature in MainActivity

Checklist

Common

  • I am ran the app before creating PR
  • I am ran all tests before creating PR

UI

  • I am ran the app for visual checks.

Logic

  • I am tested basic app functionality.

Changes

Describe all changes here:

  • First;
  • Second;
  • Third;
  • ...

Comments

Describe all additional information here.

1.Added:
- play core library dependency in app/build.gradle
- flexible in-app update feature in MainActivity
@fartem fartem self-requested a review May 7, 2021 04:35
@fartem fartem linked an issue May 7, 2021 that may be closed by this pull request
@@ -27,6 +38,9 @@
public static final String URI_FROM_EXTERNAL_APP
= "com.smlnskgmail.jaman.hashcheckerlite.URI_FROM_EXTERNAL_APP";

private static final int MY_REQUEST_CODE = 1;
Copy link
Member

Choose a reason for hiding this comment

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

Please rename MY_REQUEST_CODE to REQUEST_APP_UPDATE.

Snackbar snackbar =
Snackbar.make(
findViewById(android.R.id.content).getRootView(),
"An update has just been downloaded.",
Copy link
Member

Choose a reason for hiding this comment

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

You need to move all strings in UI to strings.xml file in resources. I'm translating this sentences to other languages.

1. Added:
- strings from UI to strings.xml

2. Updated:
- MY_REQUEST_CODE to REQUEST_APP_UPDATE in MainActivity
@fartem fartem merged commit 316cd55 into hash-checker:master May 7, 2021
@vipozdn vipozdn deleted the feature branch May 7, 2021 07:24
@fartem
Copy link
Member

fartem commented May 7, 2021

@vipozdn I'm added you to Contributors list in README.md.

@vipozdn
Copy link
Contributor Author

vipozdn commented May 7, 2021

@vipozdn I'm added you to Contributors list in README.md.

That's cool! Tnx!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add In-app update feature
2 participants