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

Feature/messages #100

Open
wants to merge 8 commits into
base: master
Choose a base branch
from
Open

Feature/messages #100

wants to merge 8 commits into from

Conversation

friedger
Copy link
Contributor

@friedger friedger commented Sep 21, 2018

This PR

  • adds an action messagefrom to the pet contract
  • adds a new table to postgres messages
  • updates filtering/watching for messagefrom
  • adds a MessageBoard to the home screen that shows that last 3 messages from any monster.
  • adds a button to send a message to the home screen.

Towards #91

bildschirmfoto von 2018-10-15 23-28-00

bildschirmfoto von 2018-10-15 23-27-45

@friedger friedger mentioned this pull request Oct 15, 2018
3 tasks
@leordev
Copy link
Member

leordev commented Oct 16, 2018

Just put this info:

  • PetTypeId.png
  • "I am so happy to be there."
  • Bubble @ October 15th 2018, owned by leordev

(message, pet name, pet type id, created at, current owner)

I will add proper images, cards, and boxes around it! Nice job @friedger

@friedger
Copy link
Contributor Author

@leordev Now it looks like this
bildschirmfoto von 2018-10-16 12-40-29

I am not sure whether the time zone is correctly translated. I am using parseZone, is that correct?

@leordev
Copy link
Member

leordev commented Oct 17, 2018

Nice @friedger

I will hold it because of #89 - I need to circle with infra to update our fullnode, and I need to update the demux reader version!

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.

2 participants