Skip to content
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

Digital Objects #1543

Open
wants to merge 5 commits into
base: staging
Choose a base branch
from
Open

Digital Objects #1543

wants to merge 5 commits into from

Conversation

andrewrothman
Copy link
Contributor

Adds digital object support to Omniwallet.

Also includes testnet fixes.

Copy link
Contributor

@achamely achamely left a comment

Choose a reason for hiding this comment

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

@andrewrothman see my two comments below.
also, what do you think about making the object interface 'hidden' by default and only loading and displaying it after a user receives or holds an object token

@@ -32,8 +33,6 @@ def send_form_response(response_dict):
else:
response_status='invalid pubkey'
pubkey=None

print response_dict
Copy link
Contributor

Choose a reason for hiding this comment

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

@andrewrothman please do not remove this, it is actually used when checking user reports of tx issues to verify what the api was received

@@ -396,7 +396,9 @@
"OPTIONS":"Wallet Options",
"OPTIONSTOGGLE":"Toggle Options",
"PORTFOLIO":"Portfolio Composition by value",
"WALLETID":"Wallet ID"
"WALLETID":"Wallet ID",
Copy link
Contributor

Choose a reason for hiding this comment

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

@andrewrothman please make sure to update all the lang files in the local folder.
also make sure to update the fallback lang file https://github.com/OmniLayer/omniwallet/blob/master/www/js/lang/en-US.js

@dexX7
Copy link
Member

dexX7 commented Feb 1, 2018

Hey @andrewrothman, what's the status of this?

@OmniLayer OmniLayer deleted a comment from puregold1958 Aug 23, 2018
@OmniLayer OmniLayer deleted a comment from puregold1958 Aug 23, 2018
@OmniLayer OmniLayer deleted a comment from dexX7 Aug 23, 2018
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.

3 participants