search

bottlesdevs / programs

Repository for programs installation
72 stars 76 forks source link

Update fusion360.yml #297

Closed DakJack closed 9 months ago

DakJack commented 9 months ago

Adjusted URL for installer.

alextrical commented 9 months ago

This is a duplicate of #296

OzzieOrca commented 9 months ago

@alextrical yes, but https://github.com/bottlesdevs/programs/pull/296 didn't get merged. It was automatically closed by https://github.com/bottlesdevs/dependencies/pull/220 which added the newly required "WebView2 Runtime" but that didn't include the url change.

alextrical commented 9 months ago

@OzzieOrca Agreed, the WebView2 was intended to be suplimental, not instead of. The WebView2 is also in the Dependency repo, as it is a Dependency. Slightly confused as to why it being merged shut down a related application that requires it as dependency.

There are a few more tweaks needed to get the software to let the user Login post the January Update/Rebrand.

I have a working Yaml, bar one additional script to create a .desktop file to handle the Mimetype. Unfortunately as of yet there has been no communication from the community as to if it's possible to: Create a cuatom .desktop file on the host Register a Mimetype on the host Check the minimum version of Wine being installed on. Wine 8.14 is now the minimum supported for this application

mirkobrombin commented 9 months ago

Do you want to keep working on this before merging?

alextrical commented 9 months ago

@mirkobrombin Could you please merge in the current change, and I will get a fully tested PR made tomorrow, along with the scripts required to be run by the end users to get the Mimetype redirection working to allow login.

Also please accept my apologies for getting the PR closed here. I was lax in my terminology. It should have said it was a suplimental add-on, instead of a fix

mirkobrombin commented 9 months ago

I see you mentioned that https://github.com/bottlesdevs/programs/pull/299 fixes this. Should I close this and check the other?

alextrical commented 9 months ago

@mirkobrombin yes if that is possible please close this and merge #299 as it contains this PR's Fix alongside additional fixes to make the ability to login functional, since the legacy methods assumed here no longer work, post the January update