Add a "Republish" feature to the data overlay #90
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
The profile and follow list tabs in the data overlay now have subtle "Republish" text buttons (if you're logged in and the owner of that profile). Pressing will take that event and try to publish it to your relays. With my own profile right now, publishing fails for most relays. Not cool. I need to check if those are dead by now, which seem to happen somewhat frequently. And there's an idea for another feature: helping users keep a clean relay list. To keep users informed, I added a notification system. You'll get notifications slide in on the top right of the screen. They update as the publish status changes. They appear on success after 5s. Ones with errors stay around until you click them off. So this is a helpful feature, but it's also a bit messy. Really needs more testing and polish.
✅ Deploy Preview for regal-crostata-f23de2 ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Add editing of relays. The profile edit page now has a second tab for relays. Just tested it and it works well. One problem I have is that the site gets its list of relays from Alby. But when post a new relay list event, Alby doesn't catch it right now. So the site still uses the old relay list. Not sure how to fix that yet.
|
Also just pushed a new page for editing your own relays. Half of mine have died in the mean time. So editing works, it's just that the site gets its list of clients from Alby, and Alby doesn't update that list right away when I publish a new event. So I refresh the page and still have my old list of relays. So basically I cannot relay on the extension data to be the latest, it should always be backed by loading from relays as well. |
|
Just did a bunch of testing. Still feels like some rough edges in a few places, but going to YOLO merge this and fix the rest little by little. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
The profile and follow list tabs in the data overlay now have subtle "Republish" text buttons (if you're logged in and the owner of that profile). Pressing will take that event and try to publish it to your relays.
With my own profile right now, publishing fails for most relays. Not cool. I need to check if those are dead by now, which seem to happen somewhat frequently. And there's an idea for another feature: helping users keep a clean relay list.
To keep users informed, I added a notification system. You'll get notifications slide in on the top right of the screen. They update as the publish status changes. They appear on success after 5s. Ones with errors stay around until you click them off.
So this is a helpful feature, but it's also a bit messy. Really needs more testing and polish.
Might also be helpful if the UI points out when there are multiple versions of events found, so users don't have to manually check. But that's probably best to add when the functionality is more stable.
Check the preview
Closes #84