Feat/create chat screen component#296
Open
mariocodecr wants to merge 7 commits into
Open
Conversation
mariocodecr
force-pushed
the
feat/createChatScreenComponent
branch
from
015296c to
ff87d61
Compare
Contributor
|
Hello sir @mariocodecr, you should to use the bubble chat components created on this issue #298 please update this one. |
mariocodecr
force-pushed
the
feat/createChatScreenComponent
branch
from
ff87d61 to
be95401
Compare
diegoTech14
reviewed
Jun 19, 2026
| type="button" | ||
| onClick={onSendPayment} | ||
| disabled={disabled} | ||
| aria-label="Enviar pago" |
Contributor
There was a problem hiding this comment.
Please, use the Button component, not the general button tag
diegoTech14
reviewed
Jun 19, 2026
| <button | ||
| type="submit" | ||
| disabled={!canSend} | ||
| aria-label="Enviar mensaje" |
Contributor
There was a problem hiding this comment.
Please, use the Button component, not the button tag
diegoTech14
reviewed
Jun 19, 2026
| ); | ||
| } | ||
|
|
||
| export function ChatMessageBubble({ |
Contributor
There was a problem hiding this comment.
This component is great, but is part of this issue: #280 I will give a chance of couple of hours to get an update, if not, I'll reassign to you because you have the solution
mariocodecr
force-pushed
the
feat/createChatScreenComponent
branch
from
3037e0c to
ede21a1
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
📝 feat: add ChatScreen component for P2P chat
🛠️ Issue
📖 Description
Implements the full chat screen between two wallet counterparts. The screen orchestrates three message types (text, payment, request) in chronological order, groups them by day, and ships with a bottom input bar that includes a "Pay" shortcut.
Per the issue instructions:
This iteration was rewritten to reuse the components recently merged into main instead of duplicating them.
✅ Changes made
🖼️ Media (screenshots/videos)
📜 Additional Notes