Skip to content
This repository was archived by the owner on Aug 30, 2022. It is now read-only.

first version commit #1

Merged
merged 23 commits into from
Apr 30, 2019
Merged

first version commit #1

merged 23 commits into from
Apr 30, 2019

Conversation

anguyenqd
Copy link
Contributor

No description provided.

@anguyenqd anguyenqd requested a review from GonnaGitYou April 24, 2019 18:13
@@ -0,0 +1,13 @@
<?xml version="1.0" encoding="UTF-8"?>
<project version="4">
<component name="ProjectModuleManager">
Copy link
Contributor

Choose a reason for hiding this comment

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

you should probably add this to the .gitignore

android:layout_width="wrap_content"
android:layout_height="wrap_content"
android:orientation="vertical"
app:layout_constraintGuide_begin="395dp" />
Copy link
Contributor

Choose a reason for hiding this comment

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

You may want to change these to percentages.

android:id="@+id/edt_from_account"
android:layout_width="0dp"
android:layout_height="wrap_content"
android:hint="Your account name"
Copy link
Contributor

Choose a reason for hiding this comment

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

We should use string resources for all the hard-coded strings.

android:id="@+id/btn_check_balance"
android:layout_width="0dp"
android:layout_height="0dp"
android:text="CHECK"
Copy link
Contributor

Choose a reason for hiding this comment

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

Maybe we should say "CHECK BALANCE"....or something similar so they know what they are checking.

app:layout_constraintStart_toStartOf="@+id/guideline4"
app:layout_constraintTop_toTopOf="@+id/guideline3" />

<Button
Copy link
Contributor

Choose a reason for hiding this comment

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

Can we add a clear button to clear the logging display?

app:layout_constraintStart_toStartOf="@+id/guideline"
app:layout_constraintTop_toBottomOf="@+id/edt_amount" />

<EditText
Copy link
Contributor

Choose a reason for hiding this comment

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

The red and green don't seem to show up well on the gray background. Maybe we could experiment with some other color combos.

*/
public static String getBackendErrorMessageFromResponse(@NonNull RPCResponseError error) {
StringBuilder detail = new StringBuilder();
if (error.getError().getDetails().size() > 0) {
Copy link
Contributor

Choose a reason for hiding this comment

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

Use isEmpty() on collection.

import one.block.eosiojavarpcprovider.implementations.EosioJavaRpcProviderImpl;
import one.block.eosiosoftkeysignatureprovider.SoftKeySignatureProviderImpl;
import one.block.eosiosoftkeysignatureprovider.error.ImportKeyError;

Copy link
Contributor

Choose a reason for hiding this comment

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

Let's add a comment for this class describing how we are using the libraries to create, sign, and send a transaction.

Copy link
Contributor

@GonnaGitYou GonnaGitYou left a comment

Choose a reason for hiding this comment

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

Please see comments.

@GonnaGitYou GonnaGitYou merged commit 929cf8b into develop Apr 30, 2019
@GonnaGitYou GonnaGitYou deleted the feature/finish_first_version branch April 30, 2019 13:35
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.

2 participants