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

1.40.0 Update #126

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

1.40.0 Update #126

wants to merge 11 commits into from

Conversation

qqrz997
Copy link
Contributor

@qqrz997 qqrz997 commented Dec 14, 2024

only thing worth noting is that there are two public methods for StretchableObstacle but SetAllProperties looks like the new init.

i also just removed some inconsistencies in the references, and added the new BeatSaber.GameSettings

P.S. i would wait before cores are done before merging, SongCore for example is getting rewritten and a lot of their APIs are going to change, i've already updated one here, aaaand i've done many changes in CustomNotes so i'll have to update that part too

@qqrz997
Copy link
Contributor Author

qqrz997 commented Dec 14, 2024

CustomNotes type updated now, this will work with the version i'll upload for 1.40.0

@kinsi55
Copy link
Owner

kinsi55 commented Dec 14, 2024

Unrelated but why was the name changed? Seems entirely unnecessary

@qqrz997
Copy link
Contributor Author

qqrz997 commented Dec 15, 2024

it bothered me? i don't have a real answer, but it's no matter now that everything else has changed too

@Smooth-Butter Smooth-Butter mentioned this pull request Dec 15, 2024
@qqrz997
Copy link
Contributor Author

qqrz997 commented Dec 17, 2024

alright, seems like everything is ready, i'm fine with merging

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