Replying to Avatar Silberengel

relay doens't work

Hello npub1896p07z8xngpct5ma00mdrad4gqfnwfwdqcl706wrm25ajynahhs27x5ge

The ngit repo for nips contains wss://purplepage.es, but that relay doesn't accept normal notes. It's more for profile data, so it should maybe be removed from the list.
39
purple.co 1y ago

do you see 3 or 4 relays after ngit pull / list ?

Reply to this note

Please Login to reply.

Discussion

Avatar
Silberengel 1y ago

nostr:npub15qydau2hjma6ngxkl2cyar74wzyjshvl65za5k5rl69264ar2exs5cyejr Why does it ask me for the naddr every time?

Avatar
DanConwayDev 1y ago

Its because the 'earliest unique commit' doesnt match the initial commit. So it can't find it automatically.

Ngit doesn't store the selected repository yet so it tries to find it each time.

Avatar
Silberengel 1y ago

Ah, okay.

39
purple.co 1y ago

Can we add naddr to yaml and use if not empty?

Avatar
DanConwayDev 1y ago

naddr is encoded kind, npub, identifer and optional relays. The missing piece in maintainers.yaml is identifier. we could potentially add that.

39
purple.co 1y ago

how ? specs ? i think is ok for maintainers.yaml to point to right address/identifier ( like git remote ? ) and use/cache/change.

Avatar
DanConwayDev 1y ago

like this:

```

identifier:

gitworkshop

maintainers:

- nostr:npub15qydau2hjma6ngxkl2cyar74wzyjshvl65za5k5rl69264ar2exs5cyejr

relays:

- wss://relay.damus.io

- wss://nos.lol

- wss://relay.nostr.band

```

maintainers.yaml is not in the nip34 spec. If we sort out the clone UX, I think the file is only really useful for a minority of repositories.

39
purple.co 1y ago

how about if is part of rust-nostr-ngit sdk ? Maybe git extension can be put in crate as a extension. If is used every time I try pull or list, is not a "minority".

Avatar
DanConwayDev 1y ago

You are right in that ngit currently looks for the maintainers.yaml every time to identify the maintainers. What I mean is that it doesn't need to. Ideally when a user first uses ngit on a repository (either by cloning the repo using ngit, a feature not yet built, or throug `ngit list`) they should choose a pubkey:identifer to trust.

39
purple.co 1y ago

I think the idea is not to enter naddr every time, and have that store/compute. Is like url for a remote after clone, just a reference that you use, and not have to input after , for any operation.

Avatar
Silberengel 1y ago

I can't figure out how to pull the changes. I see

finding proposals...

✔ 3 events wss://relay.damus.io/

✘ error wss://relay.nostr.ai/ connection timeout

✔ 0 events wss://relay.snort.social/

✔ 0 events wss://nostr.einundzwanzig.space/ ? all proposals ›

Edited maintainers file

2024.05.16-10.17.43

Burning an npub

But when I try to do ngit pull, I get

finding proposal root event...

✔ 3 events wss://relay.damus.io/

✘ error wss://relay.nostr.ai/ connection timeout

✘ error wss://relay.snort.social/ connection timeout

✔ 0 events wss://nostr.einundzwanzig.space/

Error: cannot find a proposal root event associated with the checked out branch name

The branch is master.

Avatar
DanConwayDev 1y ago

Which proposal did you select? Did you check it put as a branch?

Avatar
Silberengel 1y ago

It turned out that the proposal I was trying to pull was corrupt because of a missing file-ending. I thought the errors were some sort of conflict, but they weren't.