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.

Sign up for GitHub

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

Jump to bottom

Do signature checking in the background #10

Open
jb55 opened this issue Jul 28, 2022 · 3 comments · May be fixed by #2121
Open

Do signature checking in the background #10

jb55 opened this issue Jul 28, 2022 · 3 comments · May be fixed by #2121
Labels
enhancement Improvement security

Comments

@jb55
Copy link
Collaborator

jb55 commented Jul 28, 2022

We should have a work queue to process things in the background, like preloading PFPs, etc. We shouldn't see anything popping in.
@jb55
Copy link
Collaborator Author

jb55 commented Aug 8, 2022

This is done for PFPs now but we'll probably need it for processing signatures since that seems to be blocking the thread when scrolling on slower devices

@jb55 jb55 changed the title Load more things in the background Do signature checking in the background Aug 8, 2022
@jb55
Copy link
Collaborator Author

jb55 commented Aug 8, 2022

We need to do a bunch of processing in the background when we received a post. I've noticed the parsing content for mentions, etc can be somewhat time consuming as well. so perhaps this call all be done right away in a work queue.

@bebopkid bebopkid added the enhancement Improvement label Dec 22, 2022
suhailsaqan added a commit to suhailsaqan/damus that referenced this issue Jun 4, 2023
@suhailsaqan
# This is a combination of 18 commits.
41b7bfb 
# This is the 1st commit message:

add separate_images and separate_invoices

# This is the commit message #2:

add only_text setting to NoteContentView

# This is the commit message #3:

add only_text to dm_options

# This is the commit message #4:

add timestamp for dms

# This is the commit message #5:

add filter_content func to classify non-text content

# This is the commit message #6:

add ChatBubbleShape shape

# This is the commit message damus-io#7:

add Invoice view for dms

# This is the commit message damus-io#8:

add Image view for dms

# This is the commit message damus-io#9:

make different content of an event appear in split and change design of chat bubble

# This is the commit message damus-io#10:

add groupEventsByDateAndPubkey function

# This is the commit message damus-io#11:

improve InputField and Footer

# This is the commit message damus-io#12:

remove unused code

# This is the commit message damus-io#13:

add DM Header and reorganize DMChatView body

# This is the commit message damus-io#14:

make Messages appear in scrollview based on grouping

# This is the commit message damus-io#15:

add functions to set height of input field

# This is the commit message damus-io#16:

remove unused code

# This is the commit message damus-io#17:

add nwc paste button

# This is the commit message damus-io#18:

Fix CI tests

Closes: damus-io#1204
@jb55 jb55 mentioned this issue Sep 5, 2023
Closed
@jb55 jb55 linked a pull request May 6, 2024 that will close this issue
Update nostrdb, switch to local relay model #2121
Draft
3 tasks
@jb55
Copy link
Collaborator Author

jb55 commented May 6, 2024 •
edited
Loading

This is fixed by nostrdb +

@alltheseas alltheseas moved this to Backlog in Damus Roadmap 🛣️ May 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement Improvement security
Projects
Damus Roadmap 🛣️
Status: Backlog
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Update nostrdb, switch to local relay model damus-io/damus
3 participants
@jb55 @bebopkid @alltheseas