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

Fungible Tokens & Direct Tipping #838

Closed
wants to merge 7 commits into from

Conversation

davidyuk
Copy link
Member

@davidyuk davidyuk commented Oct 8, 2020

the rebased version of #810 with applied #832
I have squshed the overlapping changes, reducing the commit amount that have simplified rebasing. The diff is almost the same as in the base branch, except that I have extracted #836, you can compare branches by git diff feature/Fungible-Tokens-Direct-Tipping-3 feature/fungible-token-featureflagged. I'm going to test this and check the iframe wallet state.

davidyuk and others added 7 commits October 8, 2020 16:52
consistently use amount_AE
update contract
replace local contract with github version

replace local contract with github version
fixup 1
fix atoms to ae
adjust tokenInfo route
create or change allowance fix

fix change allowance
show all tokens for tip
Fixup SendTip
fixup unshift
fixup utils/aeternity
fixup config/constants
fixup send comment type
fixup AeAmount
fixup AeAmountFiat
fixed token balances vuex usage

fixup
fixup show token balances
fixed topics amount

show tokens amount only without iframe wallet
persist token information in state
fixup token balances
fixup RightSectionWallet
Fixup AeInputAmount
fixup TipRecord
fixup token selector
@davidyuk davidyuk marked this pull request as ready for review October 9, 2020 04:42
@davidyuk
Copy link
Member Author

davidyuk commented Oct 9, 2020

I'm closing this one because of the branch name the deployment is broken

@davidyuk davidyuk closed this Oct 9, 2020
@davidyuk davidyuk deleted the feature/Fungible-Tokens-Direct-Tipping-3 branch October 9, 2020 04:51
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