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

Scrapper no longer needed #22

Open
devadathanmb opened this issue Oct 28, 2023 · 3 comments
Open

Scrapper no longer needed #22

devadathanmb opened this issue Oct 28, 2023 · 3 comments
Labels
critical-bug help wanted Extra attention is needed

Comments

@devadathanmb
Copy link

devadathanmb commented Oct 28, 2023

Hey there. KTU recently completely shifted their website to React and started making use of APIs for almost everything in their page. So this project might require a rewrite at some parts. Good thing is that we don't need to go through the pain of scraping anymore.

I have hacked together some code and tried to make a replica of this in typescript (see here) although I still haven't tested it out completely yet.

@AJAYK-01
Copy link
Owner

Interesting to see that KTU finally updated their website.

I am no longer a student of KTU, hence not actively updating this project. However any contributions to this project are welcome!

@AJAYK-01 AJAYK-01 added help wanted Extra attention is needed critical-bug labels Oct 30, 2023
@devadathanmb
Copy link
Author

They have indeed tried to make this website (the current one in react) live several times before this I guess but after a while they rolled back to the old version. Now I guess it's going to stay.

This project was basically my motivation for writing my own version for the updated website. I guess only the scrapper part needs to be rewritten and rest everything could stay.

@AJAYK-01
Copy link
Owner

They have indeed tried to make this website (the current one in react) live several times before this I guess but after a while they rolled back to the old version. Now I guess it's going to stay.

This project was basically my motivation for writing my own version for the updated website. I guess only the scrapper part needs to be rewritten and rest everything could stay.

Yep, replacing scrapper with an api integration would make this work as expected.

Also I checked out the bot you created, and I saw the ui and buttons for viewing notifications, which is possible due to the new api, so I think the api could further create more optimisations and better UX for this bot as well.

@AJAYK-01 AJAYK-01 pinned this issue Nov 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
critical-bug help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

2 participants