Skip to content

database(indexeddb): Add support for NIP62#1212

Open
ndungudedan wants to merge 2 commits intorust-nostr:masterfrom
ndungudedan:indexeddb_support_nip62
Open

database(indexeddb): Add support for NIP62#1212
ndungudedan wants to merge 2 commits intorust-nostr:masterfrom
ndungudedan:indexeddb_support_nip62

Conversation

@ndungudedan
Copy link
Contributor

Handle requests to vanish by rejecting events from vanished pubkeys and wiping all existing events related to them.

Closes: #1199

Checklist

Handle requests to vanish by rejecting events from vanished pubkeys and wiping all existing events related to it.

Closes: rust-nostr#1199
Comment on lines -37 to +38
const ALL_STORES: [&str; 1] = [EVENTS_CF];
const VANISHED_PUBLIC_KEYS_CF: &str = "vanished-public-keys";
const ALL_STORES: [&str; 2] = [EVENTS_CF, VANISHED_PUBLIC_KEYS_CF];
Copy link
Member

Choose a reason for hiding this comment

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

@yukibtc why ALL_STORES doesn't contains EVENTS_SEEN_BY_RELAYS_CF? I think it should contains it

Copy link
Member

Choose a reason for hiding this comment

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

The EVENTS_SEEN_BY_RELAYS_CF is no longer used. The const is used just for dropping the store in migrations

Comment on lines +26 to +41
## Unreleased

### Breaking changes

### Changed

### Added

- Support NIP-62 RequestToVanish event kind (https://github.com/rust-nostr/nostr/pull/1212)

### Fixed

### Removed

### Deprecated

Copy link
Member

Choose a reason for hiding this comment

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

We only need Added here

Suggested change
## Unreleased
### Breaking changes
### Changed
### Added
- Support NIP-62 RequestToVanish event kind (https://github.com/rust-nostr/nostr/pull/1212)
### Fixed
### Removed
### Deprecated
## Unreleased
### Added
- Support NIP-62 RequestToVanish event kind (https://github.com/rust-nostr/nostr/pull/1212)

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.

database(indexeddb): Support NIP-62

3 participants