Comment: https://github.com/BlowaterNostr/blowater/issues/115#issuecomment-1699782866

Author: nostr:npub1dww6jgxykmkt7tqjqx985tg58dxlm7v83sa743578xa4j7zpe3hql6pdnf

Need to synchronize all seen pub keys

PR: https://github.com/BlowaterNostr/blowater/pull/124

Title: More kind 4 loading

Author: nostr:npub1dww6jgxykmkt7tqjqx985tg58dxlm7v83sa743578xa4j7zpe3hql6pdnf

PR: https://github.com/BlowaterNostr/blowater/pull/123

Title: The copy button should give feedback after clicked

Author: nostr:npub14kcnaaguwqww5cac9m2p755g8z0ugpg7zzcnczll5al86cwfj67sjk2chk

Issue: https://github.com/BlowaterNostr/blowater/issues/122

Title: The copy button should give feedback after clicked

Author: nostr:npub14kcnaaguwqww5cac9m2p755g8z0ugpg7zzcnczll5al86cwfj67sjk2chk

Issue: https://github.com/BlowaterNostr/nostr.ts/issues/18

Title: NIP6: mnemonic seed phrase

Author: nostr:npub1dww6jgxykmkt7tqjqx985tg58dxlm7v83sa743578xa4j7zpe3hql6pdnf

https://github.com/nostr-protocol/nips/blob/master/06.md

PR: https://github.com/BlowaterNostr/blowater/pull/121

Title: Fix Profile Syncing

Author: nostr:npub1dww6jgxykmkt7tqjqx985tg58dxlm7v83sa743578xa4j7zpe3hql6pdnf

PR: https://github.com/BlowaterNostr/nostr.ts/pull/17

Title: Reorg tests

Author: nostr:npub1dww6jgxykmkt7tqjqx985tg58dxlm7v83sa743578xa4j7zpe3hql6pdnf

Issue: https://github.com/BlowaterNostr/nostr.ts/issues/16

Title: Breaking Change Request: redesign updateSub's API

Author: nostr:npub1dww6jgxykmkt7tqjqx985tg58dxlm7v83sa743578xa4j7zpe3hql6pdnf

# Problem

Consider

```ts

const subID = "xxx"

const stream1 = await pool.newSub(subID, {...});

const stream2 = await pool.updateSub(subID, {...});

```

Currently, `stream1` and `stream2` are the same reference. Updating a subscription won't kill the channel created by the previous subscription.

This is an abstraction invented by `nostr.ts` since nostr protocol does not have the concept of updating an existing subscription. Our implementation close the previous subscription and creates a new subscription with the same name.

This design looks nice and convenient at first, but after several months' practice, I find this API behavior might not be desireable.

1. Hard to debug

In Blowater's production, we encountered dangling promises or starvation of channels. https://github.com/BlowaterNostr/blowater/issues/115

The root cause is still unknown but this API design should be related because Blowater did not have this problem before introducing `updateSub`

2. Unclear responsibility for channel consumers

In a real world situation, `stream1` and `stream2` are likely to be constructed in different places of the code and in different time. Since `stream1` and `stream2` are the same reference. Their consumers effectly are racing to read from the same channel which might not be desired and might cause surprising data loss from individual consumer's perspective.

3. Suprising Data for consumers

Following No.2, the reader of `stream1` might be surprised that it gets data which doesn't match its original filter.

# Proposal

1. Must

We should close the channel of `stream1` when `updateSub` is called, making `stream1` and `stream2` 2 separate references.

2. Maybe

Forbid the reuse of the same subscription id for different subscriptions and remove `updateSub`.

In this case, the caller of this API should do

```js

const stream1 = await pool.newSub("profileStream 1", {...});

const stream2 = await pool.updateSub("profileStream 2", {...});

```

Use a suffix index to indicate that it's from the same feature but for different iterations.

Comment: https://github.com/BlowaterNostr/blowater/pull/112#issuecomment-1697561765

Author: nostr:npub1dww6jgxykmkt7tqjqx985tg58dxlm7v83sa743578xa4j7zpe3hql6pdnf

Let's rework this according to #120

PR: https://github.com/BlowaterNostr/blowater/pull/120

Title: button group ui

Author: nostr:npub14kcnaaguwqww5cac9m2p755g8z0ugpg7zzcnczll5al86cwfj67sjk2chk

Some thing like

Issue: https://github.com/BlowaterNostr/blowater/issues/119

Title: Button group

Author: nostr:npub14kcnaaguwqww5cac9m2p755g8z0ugpg7zzcnczll5al86cwfj67sjk2chk

create a generic button group.

PR: https://github.com/BlowaterNostr/nostr.ts/pull/15

Title: Concurrent Testing

Author: nostr:npub1dww6jgxykmkt7tqjqx985tg58dxlm7v83sa743578xa4j7zpe3hql6pdnf

PR: https://github.com/BlowaterNostr/blowater/pull/118

Title: partially address profile sync bug

Author: nostr:npub1dww6jgxykmkt7tqjqx985tg58dxlm7v83sa743578xa4j7zpe3hql6pdnf

PR: https://github.com/BlowaterNostr/blowater/pull/117

Title: Use submodule

Author: nostr:npub1dww6jgxykmkt7tqjqx985tg58dxlm7v83sa743578xa4j7zpe3hql6pdnf

PR: https://github.com/BlowaterNostr/blowater/pull/116

Title: logo offset fix

Author: nostr:npub14kcnaaguwqww5cac9m2p755g8z0ugpg7zzcnczll5al86cwfj67sjk2chk

Issue: https://github.com/BlowaterNostr/blowater/issues/115

Title: Not able to synchronize all profiles

Author: nostr:npub1dww6jgxykmkt7tqjqx985tg58dxlm7v83sa743578xa4j7zpe3hql6pdnf

When the application loads, some profiles can't be synchronized

But if copy & paste the pub key to the search bar, the profile will load

Meaning, Blowater failed to load profiles

Comment: https://github.com/BlowaterNostr/blowater/issues/30#issuecomment-1694494595

Author: nostr:npub1dww6jgxykmkt7tqjqx985tg58dxlm7v83sa743578xa4j7zpe3hql6pdnf

We are not ready for this feature yet

PR: https://github.com/BlowaterNostr/blowater/pull/114

Title: show hex pubkeys

Author: nostr:npub1dww6jgxykmkt7tqjqx985tg58dxlm7v83sa743578xa4j7zpe3hql6pdnf

Comment: https://github.com/BlowaterNostr/blowater/pull/112#issuecomment-1694486188

Author: nostr:npub1dww6jgxykmkt7tqjqx985tg58dxlm7v83sa743578xa4j7zpe3hql6pdnf

@foodstr Is it ready to review?

Issue: https://github.com/BlowaterNostr/blowater/issues/113

Title: Logo Offset

Author: nostr:npub1dww6jgxykmkt7tqjqx985tg58dxlm7v83sa743578xa4j7zpe3hql6pdnf