Skip to content
This repository was archived by the owner on Aug 9, 2024. It is now read-only.

Better Notifications & Messaging #20

Open
wants to merge 152 commits into
base: master
Choose a base branch
from

Conversation

GStudiosX2
Copy link
Contributor

@GStudiosX2 GStudiosX2 commented Aug 1, 2024

todos

  • Better Infinite Scrolling
    At the moment it scrolls a bit funny
  • Fix duplicate ID issue for infinite scrolling ?
    I have no idea where to start on that

Summary

  • Faster loading times (Go faster. Literally. Go faster. Literally. GStudiosX2/lyntr#8)

    • Host the Lexend font CSS locally: 23f8e72
    • Remove useless preconnect links: b579cab
    • Set max cache age to 1 day: 2e589c5 broke open graph so removed for now
    • Precompress stuff via adapter options: ae20a4d
    • Preprocess scripts using vite and set appType to custom: df13aec
  • Removed DB_* and use DATABASE_URL in drizzle.config.ts (aefb592)

  • Prevent users from sending empty messages (5c82c7c)

  • Added messaging to other users that follow you back

  • Notification plays a ding animation every time you receive notifications (516bb07)

  • Fix search not displaying images etc

  • Search now has filters from:@handle (d46736d) and type:users (f800e3f)

  • Comments can now have images (dc4a832) (de4f244)

  • Reply now has a character counter (21f0cc5)

  • Clear inputs after posting (d6442f9)

  • Moved dark mode switcher into the profile button (141d01d)

Should fix #35

@GStudiosX2
Copy link
Contributor Author

Omg why is git so dumb

@GStudiosX2
Copy link
Contributor Author

Give me a second to test to make sure I didn't break anything because I had to resolve it in the web editor

@GStudiosX2
Copy link
Contributor Author

Face can you fix the conflicts I don't want to fix it xD

@CodeName-Anti
Copy link
Contributor

Something isn't right

brave_TEWJJzGuxm

@GStudiosX2
Copy link
Contributor Author

GStudiosX2 commented Aug 9, 2024

Something isn't right

That is a bug in master and it's probably fixed now but I gotta merge it first I think pr #19 caused it but not sure

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

You can post something twice by double clicking the "post" button before the first one loads
6 participants