Skip to content

fix: string -> int for ports.discovery and listener#3438

Merged
jangko merged 4 commits intostatus-im:fix-importsfrom
barnabasbusa:fix-ports-json-types
Jul 2, 2025
Merged

fix: string -> int for ports.discovery and listener#3438
jangko merged 4 commits intostatus-im:fix-importsfrom
barnabasbusa:fix-ports-json-types

Conversation

@barnabasbusa
Copy link
Copy Markdown
Contributor

No description provided.

@barnabasbusa
Copy link
Copy Markdown
Contributor Author

cc: @jangko

@jangko
Copy link
Copy Markdown
Contributor

jangko commented Jul 1, 2025

please fix the test too

 res.ports.discovery.len > 0
 res.ports.listener.len > 0

@barnabasbusa
Copy link
Copy Markdown
Contributor Author

Fixed

@barnabasbusa
Copy link
Copy Markdown
Contributor Author

Peer counts now show up on dora:
Screenshot 2025-07-01 at 20 32 50

@jangko jangko merged commit e528f5f into status-im:fix-imports Jul 2, 2025
jangko added a commit that referenced this pull request Jul 2, 2025
* feat: add admin_peers and admin ns (#3431)

* feat: add admin_peers and admin ns

* fix redundant boolean checks and import std sections

* move caps in the main block

* setup admin and quit combined into one call

* fix compile issues

* Add export marker

* Fix tests

* Restore invalid request exeception in admin_addPeer

* Chicken and egg

* oops

* fix: string -> int for ports.discovery and listener (#3438)

* fix: string -> int for ports.discovery and listener

* use int not hex

* fix test

* Add export marker

* Add comments

---------

Co-authored-by: Barnabas Busa <busa.barnabas@gmail.com>
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.

3 participants