Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Too Many Relays Warning #544

Open
alltheseas opened this issue Feb 6, 2023 · 13 comments
Open

Too Many Relays Warning #544

alltheseas opened this issue Feb 6, 2023 · 13 comments
Labels
feature New feature request good-first-issue Good for newcomers onboarding perf

Comments

@alltheseas
Copy link
Collaborator

User Story

As Damus engineer who wants to ensure Damus newbies avoid relay onboarding pitfalls, I want to warn Damus users if they are adding too many relays, so that they do not run into bandwidth, and/or machine performance issues.

Acceptance Criteria

  1. Damus users are warned when they attempt to add more than 15 relays
  2. Warning modal reads: "Warning! Adding more than 15 relays may contribute to excessive machine load, and/or excessive bandwidth usage. It is recommended you do not add more than 15 relays."
  3. User can exit with "Got it"

"You might also want to warn them that this app requires large quantity of data transactions when using mobile-data network…"
-Pomcan npub15threpkztu32y0st67u27xrrr2v5hrt848a60djh48dkx4sv33rssf7rxu
https://damus.io/note149d3x2jwm6epwgwfd7spj9aswggumjsw9255sur86zkn5etlfm0sfe6znr

"Definitely need warnings when users add too many relays"
-@jb55
https://damus.io/note1vvlmvtcsjf5djmx22dr76rvl50gqud9t2tfajxv04tpnl844fmaqvxjzpm

@alltheseas alltheseas changed the title Too Many Relays Warning [UX][Feature Request] Too Many Relays Warning [UX][Feature Request][Newbies] Feb 6, 2023
@dtdannen
Copy link

Ah so having less than 15 relays is recommended - good to know. I've been running 18 for the past week and experiencing a range of issues

@alltheseas
Copy link
Collaborator Author

alltheseas commented Mar 22, 2023

Ah so having less than 15 relays is recommended - good to know. I've been running 18 for the past week and experiencing a range of issues

Yes. Take a look at nostr.watch, to check how relays perform for you.

@alltheseas
Copy link
Collaborator Author

@BenGWeeks

@bryanmontz

related to the relay suggestions work

@alltheseas alltheseas added feature New feature request onboarding labels Jun 30, 2023
@alltheseas alltheseas changed the title Too Many Relays Warning [UX][Feature Request][Newbies] Too Many Relays Warning Dec 4, 2023
@alltheseas
Copy link
Collaborator Author

Seems to still affect some users. Warning of too many relays would be a helpful guardrail to help prevent a negative user experience for some users.

@alltheseas alltheseas added the good-first-issue Good for newcomers label Dec 4, 2023
@ser-sleepy
Copy link

image

@ser-sleepy
Copy link

image

User npub15h788cf85h6fnmm6ku8c3cfts660m2dfrkmgc6qk8wr8ut5h70rsksdmjv suggests to make a relay walkthrough part of the onboarding process.

@alltheseas
Copy link
Collaborator Author

alltheseas commented Dec 4, 2023

Thanks @ser-sleepy

There is a trade-off in onboarding: just hit "start" and post a note as quickly as possible vs doing a nostr relay tutorial, and other nostr/LN tutorials.

Maybe we could use Tipkit #1255 to prompt education in app, post onboarding, in addition to adding the too many relays warning.

@jb55
Copy link
Collaborator

jb55 commented Dec 4, 2023 via email

@alltheseas
Copy link
Collaborator Author

One thing that might be really useful is for damus to track relay stats.
It can measure the latency of pings and requests, and then automatically
disable certain relays if they are dead, slow, or unresponsive. Damus
could automatically pick the top performing relays, instead of blindly
blasting to all of them.

@dskvr does nostr watch API exist and allow for this?

@jb55
Copy link
Collaborator

jb55 commented Dec 4, 2023 via email

@ser-sleepy
Copy link

Thanks @ser-sleepy

There is a trade-off in onboarding: just hit "start" and post a note as quickly as possible vs doing a nostr relay tutorial, and other nostr/LN tutorials.

Maybe we could use Tipkit #1255 to prompt education in app, post onboarding, in addition to adding the too many relays warning.

No problem! Im thinking of a possible solution being a "tutorials" section on the menu below "merch" n above "settings" which can be "skipped" during the onboarding process if you just want to jump straight into posting and go back to it at a later time.. just an idea

@alltheseas
Copy link
Collaborator Author

alltheseas commented Dec 4, 2023

Im thinking of a possible solution being a "tutorials" section on the menu below "merch" n above "settings" which can be "skipped" during the onboarding process if you just want to jump straight into posting and go back to it at a later time.. just an idea

This is @robagreda territory - my gut feel is that any tutorial functionality should happen as the Damus user is opening a new page view, using a feature etc. Thinking about 90s style design software I could never click on skip/close fast enough on any pop-up tutorial, and I don't recall seeking a tutorial. Excellent design is needed here.

@alltheseas
Copy link
Collaborator Author

18 relays user reporting warm phone: npub18dlusgmprudw46nracaldxe9hz4pdmrws8g6lsusy6qglcv5x48s0lh8x3

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature request good-first-issue Good for newcomers onboarding perf
Projects
Status: Backlog
Development

No branches or pull requests

4 participants