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

Modify installer links to point closer to the source installer #1745

Merged
merged 2 commits into from
Dec 3, 2022

Conversation

ebr
Copy link
Member

@ebr ebr commented Dec 3, 2022

Currently a few clicks are needed to get to the source installer, and a couple more clicks to get to the installer bundle. Proposing to update links to point more closely to the source installer, and to link installer bundles directly from Github releases, at least for 2.2.3, in an attempt to smooth over first-time installation experience.

NB - if we remove the version from the filenames, we can have an always-up-to-date link to the latest release assets without having to build the docs.

ebr added 2 commits December 3, 2022 00:19
NB: if we remove the version from the zip file names, we can link
directly to assets in the latest GH release from documentation without
the need to keep the links updated
Copy link
Collaborator

@lstein lstein left a comment

Choose a reason for hiding this comment

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

Looks good. Thank you!

@lstein lstein merged commit be871c3 into invoke-ai:main Dec 3, 2022
@tildebyte
Copy link
Contributor

@ebr;

we can have an always-up-to-date link to the latest release assets without having to build the docs...

...was my original design, for all the obvious reasons, but see below

NB - if we remove the version from the filenames

We just can't. We only just added the version, because users were literally downloading and trying to install completely random stuff, and we had no way to know what they were starting with.

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