Skip to content

Commit

Permalink
TW-1469: Add ADR for this ticket
Browse files Browse the repository at this point in the history
  • Loading branch information
nqhhdev authored and hoangdat committed Mar 1, 2024
1 parent 495782e commit 5101c89
Showing 1 changed file with 20 additions and 0 deletions.
20 changes: 20 additions & 0 deletions docs/adr/0017-remove-set-state-for-reply-message.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,20 @@
# 17. Remove `setState` for reply message

Date: 2024-03-01

## Status

Accepted

## Context

- Change from `bool replyEvent` to `ValueNotifier replyEventNotifier`.
- When click on reply button, we will update `replyEventNotifier` and show the reply message.
- Remove `setState` for reply func.

## Consequences

- We're make sure for first time click on reply button, it's work well.
- We are limited to 1 use of `setState` for responding to events.
- We're migrating `Chat` to phase out funcs that use setState.
In addition to the reply message, we will continue to migrate other functions

0 comments on commit 5101c89

Please sign in to comment.