Skip to content

Switch to Njump.to#2682

Open
4G0R4 wants to merge 10 commits intostackernews:masterfrom
4G0R4:njump-to
Open

Switch to Njump.to#2682
4G0R4 wants to merge 10 commits intostackernews:masterfrom
4G0R4:njump-to

Conversation

@4G0R4
Copy link

@4G0R4 4G0R4 commented Dec 19, 2025

Description

Njump to is a proxy that redirects to a random njump instance. More info at https://github.com/dergigi/njump-to

Screenshots

image

Additional Context

Sometimes njum.me is overflooded and does not load

Checklist

Are your changes backward compatible? Please answer below:
Yes

On a scale of 1-10 how well and how have you QA'd this change and any features it might affect? Please answer below:
10

For frontend changes: Tested on mobile, light and dark mode? Please answer below:
N/A

Did you introduce any new environment variables? If so, call them out explicitly here:
No

Did you use AI for this? If so, how much did it assist you?
No

@huumn
Copy link
Member

huumn commented Dec 19, 2025

I'm not sure embeds (iframes) will work with redirects. If it does work, I'll need to review njump.to to confirm I can trust the njump sources if we're giving them a CSP hallpass.

@huumn
Copy link
Member

huumn commented Dec 19, 2025

... the embeds are janky. we shouldn't need them really and should be able to fetch nostr stuff ourselves ... it's just more code to maintain ... so if we shouldn't/can't do these embeds, it might be time to remove them

@4G0R4
Copy link
Author

4G0R4 commented Dec 20, 2025

True I did not evaluate the embeddings visualization; I simply replaced anything njump. Removing embeds (for nostr notes) could be a good move, as it forces stackers to fill the description input with some content to provide more value to the post.

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