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

Create multiple admin settings #2

Closed
islandbitcoin opened this issue Jun 1, 2023 · 11 comments · Fixed by #4
Closed

Create multiple admin settings #2

islandbitcoin opened this issue Jun 1, 2023 · 11 comments · Fixed by #4

Comments

@islandbitcoin
Copy link
Owner

islandbitcoin commented Jun 1, 2023

Two settings improvements need to be completed:

  1. Create admin settings or config file that automatically logs in the widget to anonymous (ephemeral keys). Give the user the option to change from Anonymous to NIP-07 or NIP-46 at any time on the webpage.
  2. Create admin settings to change the color scheme.

Stretch Goal (additional 500k sats):

  • Implement NIP-28 support on this widget, giving the nostri.chat admin full moderation to a public chat.

an approved PR with these changes completed will fulfill the bounty on nostrbounties.com

@Dhruv245
Copy link

Dhruv245 commented Jun 9, 2023

HI, would love to apply for this. Let me know when to start

@islandbitcoin
Copy link
Owner Author

HI, would love to apply for this. Let me know when to start

go for it 👍

@f90fwT
Copy link

f90fwT commented Jul 8, 2023

@islandbitcoin What do you think of this flow for authenticating as Anonymous by default?

20230709011715.mp4

@f90fwT
Copy link

f90fwT commented Jul 8, 2023

I was also thinking of something like this for easier configuring of the widget.

20230709014402.mp4

@f90fwT
Copy link

f90fwT commented Jul 9, 2023

@islandbitcoin I implemented all the basic things you asked for!

20230710004159.mp4

Let me know if something like this is good enough, or things you whould like to see change. I also send you a DM on nostr about your integrating with your site. I still need to do some code cleaning up ect, before making the commit/PR

@islandbitcoin
Copy link
Owner Author

This looks good! I don't see anything I would change. 👍

I don't see a nostr DM - hopefully you sent it to the correct npub!
@npub1qqqqqqrm4aehwh4t0mknaxvaqvhxdrvrnquz7a7npdgll3lezkpqq3j5a5

@f90fwT
Copy link

f90fwT commented Jul 9, 2023

Hmm that's wierd... Can confirm i sent it to the correct npub.. And i can see it on multiple clients. Do you have some other way to private message @islandbitcoin ?

@islandbitcoin
Copy link
Owner Author

hit me on my other nostr, I use Damus with this one: @npub1qqqqqq0u2gj96tdfvqymdqn739k4s0h9rzdwyegfmalv28j7a5ssh5ntu2

@f90fwT
Copy link

f90fwT commented Jul 10, 2023

@islandbitcoin PR is created with #4

@f90fwT
Copy link

f90fwT commented Jul 10, 2023

Hosted version is also updated with https://nostri-chat-hosted.f9ofwt.click/

@islandbitcoin islandbitcoin linked a pull request Jul 10, 2023 that will close this issue
@islandbitcoin
Copy link
Owner Author

I made a comment on the PR

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 a pull request may close this issue.

3 participants