Skip to content

Conversation

@not-varram
Copy link

fixed broken redirection on notifications, check issue #104

@ksucpea
Copy link
Member

ksucpea commented May 14, 2025

Not sure if you deleted your message (I don't see it anymore but I see it on gmail) about refactoring but the newest version is actually refactored and organized much better! I'm still working out some kinks before I push it to this repo, once I upload that I will take a look at this pull

@not-varram
Copy link
Author

ah yeah, I sent you an email directly insted lol, so deleted the stuff here (cuz i didnt wanna double text xp). But yeah sounds perfect! if you dont mind me asking are yo refactoring in typescript or smth or just Js but speading the files better?

I would love to help!

@ksucpea
Copy link
Member

ksucpea commented May 16, 2025

Probably will be doing both, but right now the files are still in js. I appreciate the offer for your help! I'll probably be updating this within the next week with the refactored version that you can take a look at. I'm not sure how much help I will really need but if you have anything to suggest I'll surely take a look

@not-varram
Copy link
Author

perfect, looking forward to it!

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