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
Develop #1
Develop #1
Conversation
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.
Nice PR, please work on the files I commented, also you can edit the Java snippet in the Read me file
@@ -1,6 +1,6 @@ | |||
<?xml version="1.0" encoding="UTF-8"?> | |||
<project version="4"> | |||
<component name="ProjectRootManager" version="2" languageLevel="JDK_11" default="true" project-jdk-name="JDK" project-jdk-type="JavaSDK"> |
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.
Please remove changes that arent related to the PR
@@ -4,7 +4,7 @@ | |||
<component name="GradleSettings"> | |||
<option name="linkedExternalProjectsSettings"> | |||
<GradleProjectSettings> | |||
<option name="testRunner" value="PLATFORM" /> |
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.
Please remove the changes that arent related
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.
Noted. You mean the Non-Kotlin Files?
@@ -38,10 +38,10 @@ Example in code, Kotlin: | |||
override fun onCreate(savedInstanceState: Bundle?) { | |||
super.onCreate(savedInstanceState) | |||
//activity | |||
imagePicker = ImagePicker(activity = this) |
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.
Nice, please o the same with Java example as well. Would be nice, wouldn't it 😊
Not really. Just the changes that aren't directly associated with the PR.
Some changes that were introduced by your android studio settings.
…On Mon, 26 Jul 2021, 19:55 Oyamo Brian, ***@***.***> wrote:
***@***.**** commented on this pull request.
------------------------------
In .idea/gradle.xml
<#1 (comment)>
:
> @@ -4,7 +4,7 @@
<component name="GradleSettings">
<option name="linkedExternalProjectsSettings">
<GradleProjectSettings>
- <option name="testRunner" value="PLATFORM" />
Noted. You mean the Non-Kotlin Files?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#1 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ALQSWTMVEDZWQZDU6QSEEZLTZWHRJANCNFSM5BAENWJA>
.
|
Why did you close and delete? The pr was super great |
Added Constructor overloading to make work easier while initialising from a fragment or activity