FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

changing the download links to point to Phoebus by shroffk · Pull Request #13 · ControlSystemStudio/controlsystemstudio.org · GitHub

  1. ControlSystemStudio
  2. controlsystemstudio.org

Repository navigation

changing the download links to point to Phoebus - #13

Open
shroffk wants to merge 1 commit into
mainfrom
downloads
ConversationCommits (1)ChecksFiles changed
Open

changing the download links to point to Phoebus#13
shroffk wants to merge 1 commit into
mainfrom
downloads

Conversation

shroffk commented Nov 9, 2023

Copy link
Copy Markdown
Member

removing the references to the now non functional downloads.controlsystemstudio.org ( it would still be good to know who was hosting that )

This is just the first step while we figure out how to automatically publish on github and find a solution for the nightly common builds

This comment was marked as spam.

Sign in to view

This comment was marked as spam.

Sign in to view

netlify Bot commented Nov 9, 2023 •
edited
Loading

Copy link
Copy Markdown

✅ Deploy Preview for inquisitive-malabi-c369c9 ready!

Name Link
🔨 Latest commit 1ce45a9
🔍 Latest deploy log https://app.netlify.com/sites/inquisitive-malabi-c369c9/deploys/654d2e6796bda8000951d30e
😎 Deploy Preview https://deploy-preview-13--inquisitive-malabi-c369c9.netlify.app
📱 Preview on mobile Toggle QR Code...



Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

kasemir approved these changes Nov 9, 2023
View reviewed changes

kasemir left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

No idea how this works and why "peaceful jackson" isn't happy, but certainly a good idea to remove broken links.

Instead of adding links to specific versions like ../relases/download/v4.7.2, it should also be OK to just have the ".../releases" link, so users can then pick whatever is the latest, and you don't need to update this with every release

tynanford approved these changes Nov 9, 2023
View reviewed changes

tynanford left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Netlify works by linking to github repos and the peaceful-jackson and a few of the other netlify sites are old and probably owned by Will Rogers or others. We migrated control of netlify to Kunal's site but that required using a newer version of node JS which then broke the old Netlify sites. And we can't figure out a way to get them disconnected from the github repo.

Anyways as long as inquisitive-malabi works we are good :)

Changes look good. There was a tech talk asking about downloading CS-Studio Eclipse for building .opi files from this site, should we also include a link to download the Eclipse version from github? If we have one?

shroffk commented Nov 9, 2023

Copy link
Copy Markdown
Member Author

we should build the common eclipse CS-Studio products, host them on github and then add the link for sure

georgweiss reviewed Nov 10, 2023
View reviewed changes

georgweiss left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Agree with @kasemir, link to releases should be sufficient. Maybe som text explaining that the target will always list the latest version.

tynanford mentioned this pull request Mar 12, 2024
Open
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Show hidden characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

georgweiss georgweiss left review comments
kasemir kasemir approved these changes
tynanford tynanford approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

Back | FazBrowse Home | New Git URL