support for contacts, multiple chat threads, and persistence #77

Merged
xuu merged 14 commits from multiple_chat_panes into master 6 months ago
Owner
There is no content yet.
mlctrez added 2 commits 6 months ago
mlctrez changed title from support for contacts, multiple chat threads, and persistence to WIP: support for contacts, multiple chat threads, and persistence 6 months ago
Poster
Owner

Found an issue where multiple saltyim.Clients are causing duplicate messages. working on fix

Found an issue where multiple saltyim.Clients are causing duplicate messages. working on fix
mlctrez added 1 commit 6 months ago
5f118c3037
client is now global to avoid duplicate messages
Poster
Owner

client is now a global to avoid multple clients being created when components are loaded / unloaded

client is now a global to avoid multple clients being created when components are loaded / unloaded
Owner

@mlctrez Nice! 👌 I was going to say we should only need one Client instance 😅

@mlctrez Nice! 👌 I was going to say we _should_ only need one `Client` instance 😅
prologic force-pushed multiple_chat_panes from 5f118c3037 to 5e45efb4de 6 months ago
prologic added 1 commit 6 months ago
f912059979
Update app.wasm
prologic added 1 commit 6 months ago
449e3d295c
Add pwa-dev Makefile target
prologic added 1 commit 6 months ago
a76b6b97dc
Fix pwa-dev target
xuu force-pushed multiple_chat_panes from a76b6b97dc to ad89142f8b 6 months ago
mlctrez added 1 commit 6 months ago
a5ceca9a71 merge from master
mlctrez added 1 commit 6 months ago
17df30ebb9 pickup latest changes on master for register.go
mlctrez changed title from WIP: support for contacts, multiple chat threads, and persistence to support for contacts, multiple chat threads, and persistence 6 months ago
Poster
Owner

I think this is good to go. There are a few minor UI quirks but the core functionality is there.

I think this is good to go. There are a few minor UI quirks but the core functionality is there.
xuu approved these changes 6 months ago
cache addrCache
}
func (c *Client) String() string {
Owner

looks like you dropped this. .. oh it moved lower.

looks like you dropped this. .. oh it moved lower.
if err := me.Refresh(); err != nil {
return nil, fmt.Errorf("error looking up user endpoint %s: %w", me.HashURI(), err)
log.WithError(err).Warn("error looking up user endpoint")
Owner

should this be returned? or are we just ignoring the endpoint being empty?

should this be returned? or are we just ignoring the endpoint being empty?
xuu merged commit 969a263d06 into master 6 months ago
xuu deleted branch multiple_chat_panes 6 months ago

Reviewers

xuu approved these changes 6 months ago
continuous-integration/drone/pr Build is passing
The pull request has been merged as 969a263d06.
Sign in to join this conversation.
Loading…
There is no content yet.