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

Shifted the png files of eip-823 from root to assets #1069

Merged
merged 11 commits into from
May 8, 2018

Conversation

kashishkhullar
Copy link
Contributor

@kashishkhullar kashishkhullar commented May 8, 2018

No description provided.

@eip-automerger
Copy link

Hi! I'm a bot, and I wanted to automerge your PR, but couldn't because of the following issue(s):

  • File assets/eip-823/eip-823-token-exchange-standard-visual-representation-1.PNG is not an EIP
  • File assets/eip-823/eip-823-token-exchange-standard-visual-representation-2.PNG is not an EIP

@Arachnid
Copy link
Contributor

Arachnid commented May 8, 2018

Thanks!

@Arachnid Arachnid merged commit 31bf470 into ethereum:master May 8, 2018
Arachnid pushed a commit to Arachnid/EIPs that referenced this pull request Jul 19, 2018
* epi drafted

* minor changes made

* minor changes made

* changes to sender interface and added diagrams

* added diagrams in the markdown

* updated the diagrams

* update the headers

* made changes suggested by @Arachnid

* added png to assets folder

NOTE: After the successful exchange the contract on right owns some tokens of the contract on the left.

#### Exchanging And Spending Tokens

![token-exchange-standard-visual-representation-2](https://raw.githubusercontent.com/kashishkhullar/EIPs/master/EIPS/token-exchange-standard-visual-representation-2.PNG)
![token-exchange-standard-visual-representation-2](https://raw.githubusercontent.com/kashishkhullar/EIPs/master/assets/eip-823/eip-823-token-exchange-standard-visual-representation-2.PNG)
Copy link
Member

Choose a reason for hiding this comment

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

@kashishkhullar you have added the files to the assets directory, but these links are pointing to a remote repo. Can you change these links to be .. /assets/eip-823/eip-823-token-exchange-standard-visual-representation-2.PNG ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I have updated the links.

Copy link
Member

Choose a reason for hiding this comment

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

Thanks!

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.

4 participants