samyoul-notes/archive/2020-05.md

# 2020-05-29

## Schedule

- [x] 14:00 BST - Nix - Fundamentals (Presentation)
   - https://meet.google.com/skq-pwau-vfp

## Issues

- [x] [Change 8-eips.md to 8-supported-specs.md](https://github.com/status-im/specs/issues/116) - `discussed`
- [x] [Core Dev Call 31 Agenda](https://github.com/status-im/pm/issues/56) - `discussed`

## Reviews

- [x] [Remove personal pronouns](https://github.com/vacp2p/specs/pull/135) - `commits`, `submitted`

---

# 2020-05-28

## Issues

- [x] [#1834 - Show pending transactions](https://github.com/status-im/status-go/issues/1834) - `research`
  - Trying to find a decentralised alternative for [my current proposal](https://github.com/status-im/status-go/issues/1834#issuecomment-634650486)
- [x] [go-ethereum#21143 - JSON_RPC eth_pendingTransactions Filtering](https://github.com/ethereum/go-ethereum/issues/21143) - `created`
- [x] [Core Dev Call 31 Agenda](https://github.com/status-im/pm/issues/56) - `read`

---

# 2020-05-27

## Issues

- [x] [#1834 - Show pending transactions](https://github.com/status-im/status-go/issues/1834) - `research`, `discussed`
  - [Status relies 100% on Infura for Ethereum connection, doesn't provide `eth_pendingTransactions`](https://github.com/status-im/status-go/issues/1834#issuecomment-634560314)
  - [Filtering the pending transactions via `eth_pendingTransactions` not possible](https://github.com/status-im/status-go/issues/1834#issuecomment-634567799)
  - [Proposal for `eth_pendingTransactions` infrastructure](https://github.com/status-im/status-go/issues/1834#issuecomment-634650486)
  - [Filtering pending transactions possible via OpenEthereum `parity_pendingTransactions`](https://github.com/status-im/status-go/issues/1834#issuecomment-634731429)
  - [Pending transaction pool has 93,000 transactions. Too many for a little light node.](https://github.com/status-im/status-go/issues/1834#issuecomment-634783187)

---

# 2020-05-26

## Issues

- [x] [#1834 - Show pending transactions](https://github.com/status-im/status-go/issues/1834) - `research`
  - Efficiency research
    - [ethereum.stackexchange](https://ethereum.stackexchange.com/questions/83742/is-it-possible-to-filter-pendingtransactions)
    - [stackoverflow](https://stackoverflow.com/questions/62031380/is-it-possible-to-filter-ethereum-pendingtransactions)

## Pulls

- [x] [Structural Feedback](https://github.com/status-im/go-maven-resolver/pull/1) - `submitted`, `commits`, `discussed`, `merged`

## Reviews

- [x] [Tool for resolving Java Maven dependencies](https://github.com/status-im/go-maven-resolver) - `feedback`
- [x] [Town Hall #58 - May 25, 2020](https://www.youtube.com/watch?v=VAvayRLS6GY) - `watched`

---

# 2020-05-25

## Issues

- [x] [#1937 - Compress public key for chat](https://github.co/status-im/status-go/issues/1937) - `discussed`

## Pulls

- [x] [status-im/spec#130 - Update/sequence specs](https://github.com/status-im/specs/pull/130) - `merged`
- [x] [[vacp2p/spec#128 - Waku Confirmations Enabled and Light Node Field Descriptions](https://github.com/vacp2p/specs/pull/128) - `merged`

---

# 2020-05-22

## Schedule

- [x] 14:00 BST - Compressed Key Discussion
   - https://meet.google.com/hqw-nsfk-xoq
   - **Notes**
     - https://github.com/status-im/status-go/issues/1937
     - https://www.w3.org/TR/did-core/
     - https://github.com/status-im/status-react/pull/10703#issuecomment-632756251

## Pulls

- [x] 🎆🥳🎉 [status-im/spec#114 - Update/waku replace](https://github.com/status-im/specs/pull/114) - `merged` 🎆🥳🎉
- [x] [status-im/spec#129 - Added release information to changelogs](https://github.com/status-im/specs/pull/129) - `submitted`, `merged`
- [x] [status-im/spec#130 - Update/sequence specs](https://github.com/status-im/specs/pull/130) - `submitted`
- [x] [vacp2p/spec#128 - Waku Confirmations Enabled and Light Node Field Descriptions](https://github.com/vacp2p/specs/pull/128) - `resubmitted`
  - Some merge commits weren't signed and it caused a block on the merge. I needed to rebase and squash the commits in the branch and resign.
  - I've added auto-sign so that I will always be asked to sign commits

## Reviews

- [x] [#1982 - Load identicon & alias in chat.](https://github.com/status-im/status-go/pull/1982) - `approved`

---

# 2020-05-21

## Issues

- [x] [status-im/spec#126- Remove/Replace Whisper RPC shh_ and shhext_ methods](https://github.com/status-im/specs/issues/126) - `closed`
- [x] [#1981 - Waku host has special behaviour if a peer has `confirmations-enabled` set to true](https://github.com/status-im/status-go/issues/1981) - `created`
- [x] [status-im/spec#127 - Re-sequence spec new numbers](https://github.com/status-im/specs/issues/127) - `created`
- [x] [#1937 - Key compression initial draft](https://github.com/status-im/status-go/issues/1937) - `discussed`

## Pulls

- [x] [status-im/spec#114 - Update/waku replace](https://github.com/status-im/specs/pull/114) - `discussed`, `commits`
- [x] [vacp2p/spec#128 - Waku Confirmations Enabled and Light Node Field Descriptions](https://github.com/vacp2p/specs/pull/128) - `discussed`, `commits`

---

# 2020-05-20

## Issues

- [x] [#123 - Clarification on Status node roles](https://github.com/status-im/specs/issues/123) - `created`
- [x] [#125 - Update 'whispermail' to `wakumail`](https://github.com/status-im/specs/issues/125) - `created`
- [x] [#126- Remove/Replace Whisper RPC shh_ and shhext_ methods](https://github.com/status-im/specs/issues/126) - `created`

## Pulls

- [x] [status-im/spec#114 - Update/waku replace](https://github.com/status-im/specs/pull/114) - `discussed`, `commits`, `completed`
- [x] [vacp2p/spec#128 - Waku Confirmations Enabled and Light Node Field Descriptions](https://github.com/vacp2p/specs/pull/128) - `discussed`

---

# 2020-05-19

## Pulls

- [x] [status-im/spec#114 - Update/waku replace](https://github.com/status-im/specs/pull/114) - `discussed`, `commits`, `submitted`
- [x] [status-im/spec#120 - Fix Links](https://github.com/status-im/specs/pull/120) - `submitted`
- [x] [#1980 - Patch/dependabot protocol](https://github.com/status-im/status-go/pull/1980) - `merged`
- [x] [vacp2p/spec#128 - Waku Confirmations Enabled and Light Node Field Descriptions](https://github.com/vacp2p/specs/pull/128) - `submitted`, `discussed`

## Reviews

- [x] [status-im/spec#119 - spellcheck](https://github.com/status-im/specs/pull/119) - `feedback`
- [x] [status-im/spec#118 - Update PoW requirements](https://github.com/status-im/specs/pull/118) - `feedback`, `approved`

---

# 2020-05-18

## Schedule

- [x] 13:00 BST - Bi-weekly Core Dev Call
   - https://meet.google.com/uyn-yfrq-uka
   - [Notes](https://notes.status.im/core-dev-call_30)
   
## Issues

- [x] [Core Dev Call 30 Agenda](https://github.com/status-im/pm/issues/53) - `discussed`
- [x] [message-confirmations-field appears to be describing Message Response](https://github.com/vacp2p/specs/issues/124) - `created`
   
## Posts

- [x] [Wherefore Art Thou Mailserver - Treatise on Waku Terminology](https://discuss.status.im/t/wherefore-art-thou-mailserver-treatise-on-waku-terminology/1664) - `discussed`
- [ ] [Desktop paths forward, a choose your own adventure?](https://discuss.status.im/t/desktop-paths-forward-a-choose-your-own-adventure/1666)
- [ ] [How :desktop_computer Desktop & Mobile should connect?](https://discuss.status.im/t/how-desktop-mobile-should-connect/1668)
   
## Pulls

- [x] [#114 - Update/waku replace](https://github.com/status-im/specs/pull/114) - `discussed`, `commits`
- [x] [Patch/dependabot protocol](https://github.com/status-im/status-go/pull/1980) - `submitted`

## Reviews

- [x] [#vacp2p/specs#123 - Be consistent in usage of terms packet, message and envelope](https://github.com/vacp2p/specs/pull/123) - `Approved`

---

# 2020-05-15

## Schedule

- [x] 17:00 BST - Happy Hour, The Return
  - https://meet.google.com/vws-ihdx-iuw

## Issues

- [x] [#1978 - Consistently flaky Jenkins unit tests](https://github.com/status-im/status-go/issues/1978) - `created`
- [x] [#1977 - Your .dependabot/config.yml contained invalid details](https://github.com/status-im/status-go/issues/1977) - `read`, `addressed`

## Posts

- [x] [Proposal: Rename “mailservers” to “historynodes”](https://discuss.status.im/t/proposal-rename-mailservers-to-historynodes/1408/13) - `read`, `feedback`

## Pulls

- [x] [#114 - Update/waku replace](https://github.com/status-im/specs/pull/114) - `discussed`
- [x] [#1975 - Updated dependabot config to security only](https://github.com/status-im/status-go/pull/1975) - `merged`
- [x] [#1979 - Minor dependabot config yml format fix](https://github.com/status-im/status-go/pull/1979) - `submitted`

## Reviews

- [x] [#53 - Core Dev Call 30 Agenda](https://github.com/status-im/pm/issues/53) - `read`, `feedback`

---

# 2020-05-14

## Issues

- [x] [#1976 - Build a make-vendor-bot](https://github.com/status-im/status-go/issues/1976) - `created`
- [x] [#115 - Removal of Personal Pronouns From Specs](https://github.com/status-im/specs/issues/115) - `created`
- [x] [#116 - Change 8-eips.md to 8-supported-specs.md](https://github.com/status-im/specs/issues/116) - `created`

## Pulls

- [x] [#1949 - Waku package README.md documentation](https://github.com/status-im/status-go/pull/1949) - `merged`
- [x] [#1809 - fix: close resultsets so we don't leak them](https://github.com/status-im/status-go/pull/1809) - `commits`, `resubmitted`, `merged`
- [x] [#1972 - Updated dependabot config to assign Samyoul](https://github.com/status-im/status-go/pull/1972) - `submitted`, `merged`
- [x] [#1975 - Updated dependabot config to security only](https://github.com/status-im/status-go/pull/1975) - `submitted`
- [x] [#114 - Update/waku replace](https://github.com/status-im/specs/pull/114) - `commits`, `discussed`

---

# 2020-05-13

## Schedule

- [ ] 16:00 BST - Andre 1:1 - `postponed`
  - https://meet.google.com/rij-fgdg-xxe

## Issues

- [x] [#1968 - Documentation Linting is Ignored](https://github.com/status-im/status-go/issues/1968) - `created`
- [x] [#1970 - Remove vendored Whisper and any remain usages](https://github.com/status-im/status-go/issues/1970) - `created`

## Pulls

- [x] [#1949 - Waku package README.md documentation](https://github.com/status-im/status-go/pull/1949) - `commits`, `completed`, `resubmitted`
- [x] [#114 - Update/waku replace](https://github.com/status-im/specs/pull/114) - `commits`, `discussed`

## Reviews

- [x] [#1969 - Add support for images](https://github.com/status-im/status-go/pull/1969) - `feedback`, `approved`

---

# 2020-05-12

## Posts

- [Wherefore Art Thou Mailserver - Treatise on Waku Terminology](https://discuss.status.im/t/wherefore-art-thou-mailserver-treatise-on-waku-terminology/1664) - `created`, `submitted`
- [Status-go Database De-dithering](https://discuss.status.im/t/status-go-database-de-dithering/1665) - `created`, `submitted`

## Reviews

- [x] [Re-introduce deprecated method of requesting messages](https://github.com/status-im/status-go/pull/1967#pullrequestreview-410193812) - `approved`
- [x] [Town Hall #57 - 11 May 2020](https://www.youtube.com/watch?v=sUUh7EUnVoE) - `watched`

---

# 2020-05-08

## Schedule

- [x] 14:00 BST - Core Retro
  - https://meet.google.com/hqw-nsfk-xoq
  - [Summary](https://notes.status.im/0xw-XPU2Q9mYsn7oSLs_ag?both)

## Pulls

- [x] [#114 - Update/waku replace](https://github.com/status-im/specs/pull/114) - `commits`, `discussed`

---

# 2020-05-07

## Issues

- [x] [#1937 - Key compression initial draft](https://github.com/status-im/status-go/issues/1937) - `discussed`

## Pulls

- [x] [#114 - Update/waku replace](https://github.com/status-im/specs/pull/114) - `draft`
- [x] [#1964 - Minor Waku version subpackage message.go tidy](https://github.com/status-im/status-go/pull/1964) - `merged`

---

# 2020-05-06

## Issues

- [x] [#1937 - Key compression initial draft](https://github.com/status-im/status-go/issues/1937) - `discussed`
- [x] [status-im/specs#61 - Replace forked Whisper usage with Waku/0 in specs](https://github.com/status-im/specs/issues/61) - `read` 

## Pulls

- [x] [#118 - Change subsection titles to linkable headers](https://github.com/vacp2p/specs/pull/118) - `merged`
- [x] [#1949 - Waku package README.md documentation](https://github.com/status-im/status-go/pull/1949) - `commits`
- [x] [#1964 - Minor Waku version subpackage message.go tidy](https://github.com/status-im/status-go/pull/1964) - `submitted`

## Reviews

- [ ] [#10589 - Move icons paths](https://github.com/status-im/status-react/pull/10589)

---

# 2020-05-05

## Schedule

- [x] 14:00 BST - Andre 1:1
  - https://meet.google.com/pga-udvu-wpk
  - Points
    - [status-go#1834 - Show pending transactions](https://github.com/status-im/status-go/issues/1834)
    - [status-react#9976 - Show pending transactions](https://github.com/status-im/status-react/issues/9976)
    - [status-react#3997 - Display pending requests in transactions history](https://github.com/status-im/status-react/issues/3997)

## Pulls

- [x] [#1949 - Waku package README.md documentation](https://github.com/status-im/status-go/pull/1949) - `commits`
- [x] [#118 - Change subsection titles to linkable headers](https://github.com/vacp2p/specs/pull/118) - `submitted`

---

# 2020-05-04

## Schedule

- [x] 13:00 BST - Core Dev Call
  - https://meet.google.com/vpp-squx-vpi
  - Points
    - [Abandoned Jitsi meeting](https://meet.status.im/core-dev-call-29)
    - [Meeting Minutes](https://notes.status.im/core-dev-call-29-notes)

## Pulls

- [x] [#1949 - Waku package README.md documentation](https://github.com/status-im/status-go/pull/1949) - `commits`
- [x] [#117 - Removed light client option from Waku / Whisper diff](https://github.com/vacp2p/specs/pull/117) - `submitted`

---

# 2020-05-01

## Pulls

- [x] [#1958 - Refactor/waku tidy 1](https://github.com/status-im/status-go/pull/1958) - `completed`, `commits`, `merged`
- [x] [#1949 - Waku package README.md documentation](https://github.com/status-im/status-go/pull/1949) - `commits`, `submitted`

---