samyoul-notes/README.md

58 lines
2.7 KiB
Markdown

# 2020-12-18
## Ad Hoc
- [x] Resolve weird issue on my machine where make vendor has started throwing `modvendor: command not found`
- Caused by an Apple update https://stackoverflow.com/a/59138750/1700106
- Solution was to rename `./bash_profile` to `./zprofile`
- [x] Compile list of known technical debt in `status-go`
- [Document Here](https://notes.status.im/cLQWkUEbTQmsIvUGAyj95A)
- Based on the `//TODO` items in the code
- Gone through all directories up to `protocol`, need to finish `protocol`, all `protocol` sub directories and all remaining directories
## Schedule
- [x] 14:00 GMT - Core Retro
- https://meet.google.com/hqw-nsfk-xoq
- A review / summary of the year.
---
# 2020-12-17
## Pulls
- [x] [#2100 - Expand Local Notifications](https://github.com/status-im/status-go/pull/2100) - `commits`, `research`
- https://notes.status.im/notifications-android
- Currently have a bug that I think is caused by a the jsonMarshal func not working as intented, need to resolve this next before proceeding
---
# 2020-12-16
## Pulls
- [x] [#2051 - Profile Images](https://github.com/status-im/status-go/pull/2051) - `commits`
- Related to debugging
- Issue finally resolved by Cammellos
- [tracking of the messages would deadlock](https://github.com/status-im/status-go/blob/50b17308bde0008daf4c8365782575d4f22b1515/protocol/messenger.go#L306)
- [Making `EnvelopesMonitor.Add()` deadlock](https://github.com/status-im/status-go/blob/50b17308bde0008daf4c8365782575d4f22b1515/protocol/transport/waku/envelopes.go#L108)
- [Because `EnvelopesMonitor.handleEventEnvelopeSent()` couldn't unlock](https://github.com/status-im/status-go/blob/50b17308bde0008daf4c8365782575d4f22b1515/protocol/transport/waku/envelopes.go#L165)
- [Fix was to remove mutex lock from `Messenger.processSentMessages()`](https://github.com/status-im/status-go/blob/50b17308bde0008daf4c8365782575d4f22b1515/protocol/messenger.go#L306)
## Issues
- [x] [#11347 - Profile edit picture UI](https://github.com/status-im/status-react/pull/11347)
- [App Freezing when user sends message after changing profile image](https://github.com/status-im/status-react/pull/11347#issuecomment-746252933)
- Debugging
## Schedule
- [x] 14:00 GMT - Samuel & Andrea 1-to-1
- https://meet.google.com/rhy-trwc-ybb
- **Notes :**
- *Profile Images*
- [App Freezing when user sends message after changing profile image](https://github.com/status-im/status-react/pull/11347#issuecomment-746252933)
- *`status-go` notifications improvement*
- Adding `Notifications` field to `MessageResponse` struct and parsing in the `PublisherSignalHandler`