Remove login button - Githubissues

boom-army / boom-client

MIT License

0 stars 0 forks source link

Remove login button #47

Closed whichape closed 2 years ago

whichape commented 2 years ago

Moved login action into wallet connect #9
Fixed issue with refreshing feed after new tweet

h4rkl commented 2 years ago

Looks good. I made a change on my dev branch to re-query the feed after posting a new tweet as well, but I used the current tweet length instead of taking the first 10.

What do you / @h4rkl think we should use? But I guess ideally we want to just grab the new tweet and slot it at the top of the feed list.

Agree, definitely need to use the current tweet length @Bort-license-plate.

This is awesome @whichape.

whichape commented 2 years ago

Looks good. I made a change on my dev branch to re-query the feed after posting a new tweet as well, but I used the current tweet length instead of taking the first 10.

Cool I'll merge this in and your one can come on top