Skip to content

exchange pos#1564

Open
RCGV1 wants to merge 3 commits intomainfrom
pos-exchange
Open

exchange pos#1564
RCGV1 wants to merge 3 commits intomainfrom
pos-exchange

Conversation

@RCGV1
Copy link
Member

@RCGV1 RCGV1 commented Jan 17, 2026

What changed?

Added a text message view for exchanging positions so you can see positions that you sent or where sent back to you in response. Also made the exchange position button in text field an actual button with confirmation instead of text. I also added a notification that shows up when someone send you a position and it is configurable in the settings.

Why did it change?

The previous approach was sending a text along with the position traffic which could cause collisions. Also it is confusing since the text doesnt contain the actual position and you need to go into another tab to find it. This way you can be notified and see a log of what positions someone has sent to you.

How is this tested?

Tested extensively by sending nodes position requests

Screenshots/Videos (when applicable)

IMG_5270

IMG_5265

ScreenRecording_01-17-2026.14-32-58_1.MP4

Checklist

  • My code adheres to the project's coding and style guidelines.
  • I have conducted a self-review of my code.
  • I have commented my code, particularly in complex areas.
  • I have verified whether these changes require an update to existing documentation or if new documentation is needed, and created an issue in the docs repo if applicable.
  • I have tested the change to ensure that it works as intended.

@garthvh garthvh changed the base branch from main to 2.7.8 February 13, 2026 19:00
@garthvh garthvh changed the base branch from 2.7.8 to main February 13, 2026 19:03
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.

1 participant