search

lnbits / nostrrelay

One click nostr relay
MIT License
35 stars 7 forks source link

accounts form bug #8

Closed bitkarrot closed 1 year ago

bitkarrot commented 1 year ago

Hi - not sure if this is a work in progress or if i'm using it incorrectly but for the accounts tab in the relay extension, when i add a npub it doesn't seem to register the data in the table below.

I am using direct from main branch, and pull from main branch yesterday, April 5, 2023

Screenshot 2023-04-06 at 1 54 21 PM
motorina0 commented 1 year ago

hmm, maybe the filter UX is not the best From the screenshot you are filtering accounts that are both blocked and allowed

dharake commented 1 year ago

This issue was resolved for me by upgrading from SQLite to Postgres, hope that helps

santyr commented 1 year ago

Unfortunately, I also see this issue. regardless of toggling allowed or blocked accounts.

Running LNbits version: 0.10.8, Commit version: 8ed2422ade4fa63eaf51cbcff9a07b85a23cc1f2 using sqllite.

motorina0 commented 1 year ago

Unfortunately, I also see this issue. regardless of toggling allowed or blocked accounts.

Running LNbits version: 0.10.8, Commit version: 8ed2422ade4fa63eaf51cbcff9a07b85a23cc1f2 using sqllite.

What version of the extension do you use?

santyr commented 1 year ago

0.25

talvasconcelos commented 1 year ago

Hmm... trying to debug this one!! apparently the CRUD call sends an empty list every time! Although i tested it in SQL and it works!

Needs some more debugging... @motorina0 wouldn't it be better to call them once (from DB) and filter the array client side? Can't assing myself to this, but I'm on it!