katniny/Auride
1
0
Fork 0
Code Issues 11 Pull requests Projects Releases 2 Packages Wiki Activity Actions

Upgrade all scripts to modules #116

Merged
katniny merged 17 commits from modules-upgrade into main 2025-11-15 04:38:43 -05:00
Conversation 5 Commits 17 Files changed 102 +632 -452
katniny commented 2025-11-15 04:15:21 -05:00 (Migrated from github.com)
Copy link

Description of Changes

This PR makes literally every script on Auride a module, meaning we can use NPM modules (thanks to Vite), import functions only when needed within other scripts, more verbose errors, and any other benefits modules bring.

Mainly did this for the ability to make Auride an SPA, meaning we can navigate pages without having to refresh the page every time, which will come out... eventually.

Visual Sample

N/A, would be impossible to showcase every page and all the functionality on them.

I did my best to check every page though, and I didn't notice anything broken (unless it was already broken before this PR, that's just bad code.)

Checklist

  • I have thoroughly tested my changes to the best of my ability
    • You agree that you have tested your changes, and that you are not opening a Pull Request that you're unsure if it works.
  • I confirm I have not contributed anything that would impact Auride's licensing and/or usage
    • Auride is a commercial product that Katniny Studios can profit from. Please do not add copyrighted material to your Pull Request, however there are exceptions (e.g. our Spotify integration).
  • This Pull Request fixes a critical issue that should be reviewed and merged as soon as possible
    • This includes things such as security vulnerabilities, ways to manipulate data, etc.
  • I've read the latest CONTRIBUTING.md (last updated: November 15, 2025) and will follow the guidelines
    • Please make sure to read it, we use this as the standard when reviewing contributions, so it's in your best interest to be familiar with it!
  • I updated the version and added the update log
### Description of Changes --- This PR makes literally every script on Auride a module, meaning we can use NPM modules (thanks to Vite), import functions only when needed within other scripts, more verbose errors, and any other benefits modules bring. Mainly did this for the ability to make Auride an SPA, meaning we can navigate pages without having to refresh the page *every time*, which will come out... eventually. ### Visual Sample --- N/A, would be impossible to showcase every page and all the functionality on them. I did my best to check every page though, and I didn't notice anything broken (unless it was already broken before this PR, that's just bad code.) ### Checklist --- - [x] I have thoroughly tested my changes to the best of my ability - You agree that you have tested your changes, and that you are not opening a Pull Request that you're unsure if it works. - [x] I confirm I have not contributed anything that would impact Auride's licensing and/or usage - Auride is a **commercial** product that Katniny Studios can profit from. Please do not add copyrighted material to your Pull Request, however there are exceptions (e.g. our Spotify integration). - [ ] This Pull Request fixes a **critical** issue that should be reviewed and merged as soon as possible - This includes things such as security vulnerabilities, ways to manipulate data, etc. - [x] I've read the latest [CONTRIBUTING.md](https://github.com/katniny/auride/blob/main/CONTRIBUTING.md) (last updated: November 15, 2025) and will follow the guidelines - Please make sure to read it, we use this as the standard when reviewing contributions, so it's in your best interest to be familiar with it! - [x] I updated the version and added the update log - Unsure what this means? Please read [CONTRIBUTING.md](https://github.com/katniny/auride/blob/main/CONTRIBUTING.md).
katniny (Migrated from github.com) reviewed 2025-11-15 04:23:18 -05:00
TODO.md
katniny (Migrated from github.com) commented 2025-11-15 04:23:18 -05:00
Copy link

remove this file

remove this file
katniny (Migrated from github.com) reviewed 2025-11-15 04:31:31 -05:00
src/assets/js/ts_fas_acih.js
@ -0,0 +5185,4 @@
// check if near the bottom, if user is,
// hide the login/signup prompt
window.addEventListener("scroll", () => {
let loginPrompt = document.getElementById("notSignedIn-banner");
katniny (Migrated from github.com) commented 2025-11-15 04:31:30 -05:00
Copy link

i need to remove this. i just wanted to hide the errors for a sec so i could focus on the ones that i needed to look at

this isnt anything serious, and it works fine... unsure why it errors
thats besides the point though

i need to remove this. i just wanted to hide the errors for a sec so i could focus on the ones that i needed to look at this isnt anything serious, and it works fine... unsure why it errors thats besides the point though
miku4k commented 2025-11-15 05:02:58 -05:00 (Migrated from github.com)
Copy link

girl its literally all broken

girl its literally all broken
miku4k commented 2025-11-15 05:04:51 -05:00 (Migrated from github.com)
Copy link
image

seriously, did you test anything?

<img width="1022" height="905" alt="image" src="https://github.com/user-attachments/assets/72edf438-5a12-4275-a578-602e08d74edb" /> seriously, did you test anything?
katniny commented 2025-11-15 05:11:08 -05:00 (Migrated from github.com)
Copy link

yes, and it worked fine and magically broke in prod
lets not, im already stressed about it as is

yes, and it worked fine and magically broke in prod lets not, im already stressed about it as is
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
katniny/Auride!116
No description provided.