Skip to content

chore: update realm for latest gno compatibility - #176

Merged
D4ryl00 merged 8 commits into
gnoverse:mainfrom
D4ryl00:chore/update-gno-dep
Mar 25, 2026
Merged

D4ryl00 merged 8 commits into
gnoverse:mainfrom
D4ryl00:chore/update-gno-dep

Conversation

@D4ryl00

@D4ryl00 D4ryl00 commented Mar 24, 2026

Copy link
Copy Markdown
Contributor
  • Update package imports: p/nt/avl → p/nt/avl/v0, p/nt/ufmt → p/nt/ufmt/v0
  • Remove dependency on p/jefft0/avlhelpers (not deployed on gnoland/test12); inline listByteStringKeysByPrefix in util.gno
  • Remove r/sys/users registration requirement — unregistered users fall back to their address as username, making the realm usable on chains where the whitelist is not configured
  • Add resolveAddr() in render.gno so render paths accept raw addresses, not just registered names
  • Replace hardcoded /r/berty/social strings with a gRealmPath variable initialized at deploy time via runtime.CurrentRealm().PkgPath(), so rendered links are always correct regardless of deployment path

@D4ryl00
D4ryl00 requested a review from jefft0 March 24, 2026 16:12
@D4ryl00 D4ryl00 self-assigned this Mar 24, 2026
Signed-off-by: D4ryl00 <d4ryl00@gmail.com>
@D4ryl00

D4ryl00 commented Mar 24, 2026 •

Copy link
Copy Markdown
Contributor Author

@jefft0 I added a second commit with tests so you can run them easily with gnodev. Check the README.md.

@D4ryl00

D4ryl00 commented Mar 24, 2026 •

Copy link
Copy Markdown
Contributor Author

@jefft0 I added a second commit with tests so you can run them easily with gnodev. Check the README.md.

I would add these tests in a new PR, but I think it would be useful for you if you have them now.

@jefft0

jefft0 commented Mar 24, 2026

Copy link
Copy Markdown
Contributor

I have two users. One has a registered username "test1", the other doesn't.

image

I made them follow each other. Here is the page for test1:

image

When I click on "1 Followers", I am supposed to see the list of followers. But it is empty. Similar problem for "Following 1".

Here is the page for the unregistered user:

image

Notice that "1 Followers" and "Following 1" are gray. Can't click. Maybe this part of the code isn't using resolveAddr correctly.

D4ryl00 added 3 commits March 24, 2026 18:25
Signed-off-by: D4ryl00 <d4ryl00@gmail.com>
Signed-off-by: D4ryl00 <d4ryl00@gmail.com>
Signed-off-by: D4ryl00 <d4ryl00@gmail.com>

@jefft0 jefft0 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested in gnoweb locally. tests/test.sh passess locally.

D4ryl00 added 3 commits March 25, 2026 10:28
Signed-off-by: D4ryl00 <d4ryl00@gmail.com>
Signed-off-by: D4ryl00 <d4ryl00@gmail.com>
Signed-off-by: D4ryl00 <d4ryl00@gmail.com>
@D4ryl00
D4ryl00 merged commit 5a2917a into gnoverse:main Mar 25, 2026
2 checks passed
@D4ryl00
D4ryl00 deleted the chore/update-gno-dep branch March 25, 2026 10:32
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.

2 participants