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 to Steam" and creation of shortcuts #378

Merged
merged 72 commits into from
Jan 27, 2024

Conversation

gablm
Copy link
Member

@gablm gablm commented Jan 17, 2024

New Additions

1. Shortcut Creation

Asks for a folder and creates a .url file linking to Collapses' URL Protocol.
"-p" is be added if selected in the UI.

2. Addition of regions to Steam

Parses and writes a new shortcuts.vdf including a new shortcut for the region for every folder in Steam's userdata folder.
Also downloads the assets related to this shortcut.
If the shortcut already exists, it will be removed and added again while also checking if the md5 checksum for all assets matches and if any file needs to be downloaded.

Requirements

closes #358

@gablm gablm added this to the 1.73.x milestone Jan 17, 2024
@gablm gablm self-assigned this Jan 17, 2024
@bagusnl bagusnl requested a review from a team January 18, 2024 17:13
Copy link
Member

@bagusnl bagusnl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No code issue other than we have discussed before.
Requesting recheck by @neon-nyan due to COM interaction and encodings that is beyond my knowledge

@bagusnl bagusnl requested a review from neon-nyan January 21, 2024 14:57
Copy link
Member

@neon-nyan neon-nyan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should be looking just fine for me.
Sorry for taking so long to take a look on this

@bagusnl
Copy link
Member

bagusnl commented Jan 27, 2024

Approbal achieved

@gablm can you check on the merge conflicts? We can merge it once the conflicts are resolved
Also, you can pick if you want to merge by Squash or Merge commit

@gablm
Copy link
Member Author

gablm commented Jan 27, 2024

There doesn't seem to be any merge conflicts on my end
About merge type, I'd rather if it was by squash

@bagusnl
Copy link
Member

bagusnl commented Jan 27, 2024

I was looking at the wrong merge type, rebase causes conflict but squash and commit doesn't :derp:

Alrighty, merging by squash.
Thank you for your work! <3

@bagusnl bagusnl merged commit 3f8c4a3 into CollapseLauncher:main Jan 27, 2024
3 checks passed
@gablm gablm deleted the shortcuts branch January 27, 2024 12:38
@gablm gablm restored the shortcuts branch January 27, 2024 12:38
@gablm gablm deleted the shortcuts branch February 23, 2024 16:02
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.

[Request]: Add individual games/region to Steam
3 participants