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

Add Lazer Support #86

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

Conversation

hrfarmer
Copy link
Collaborator

@hrfarmer hrfarmer commented Oct 19, 2024

Note: I have settings.delete("osuSongsDir"); uncommented for now to make it easier to test first time setup stuff, in case you wonder why it resets after every launch

TODO

  • Figure out why song select is broken
  • Figure out how to find a custom files location (if the user moved their file location in-game)
  • Fix rich presence (add set id to song object and fix duration nonsense)
  • Make autodetection automatically happen on start
  • Update UI to new selection design
  • Fix linux lazer parsing/autodetection not working
  • Fix realm types (a lot of things that should be optional aren't)
  • Add excluded diffs to song.diffs
  • Fix song.bpm type

@hrfarmer hrfarmer marked this pull request as ready for review October 24, 2024 23:49
@hrfarmer
Copy link
Collaborator Author

There's still a few things that need to be polished here and there, but it's good enough now for some initial reviews

package.json Outdated Show resolved Hide resolved
@duduBTW
Copy link
Collaborator

duduBTW commented Nov 12, 2024

The app keeps sending me to the dir select page everytime even after importing the songs.

2024-11-12.13-41-55.mp4

@hrfarmer
Copy link
Collaborator Author

The app keeps sending me to the dir select page everytime even after importing the songs.

2024-11-12.13-41-55.mp4

erm first line of pr message...

duduBTW
duduBTW previously approved these changes Nov 13, 2024
@hrfarmer
Copy link
Collaborator Author

CURRENT ISSUE

Everything seems to work now after resolving the conflicts, EXCEPT for that I can no longer run the program whenever the lucide preprocessor is inside of the vite config. I get the same errors as this message. As such I've removed the preprocessor from the vite config temporarily.

@brw I feel you would probably know best here, do you know what could be causing this?

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.

3 participants