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

PB-1031: redirect github link to release page #1216

Merged
merged 2 commits into from
Jan 28, 2025

Conversation

ltkum
Copy link
Contributor

@ltkum ltkum commented Jan 21, 2025

The github link now leads to the specific release

Test link

@ltkum ltkum requested a review from pakb January 21, 2025 15:09
Copy link

cypress bot commented Jan 21, 2025

web-mapviewer    Run #4388

Run Properties:  status check passed Passed #4388  •  git commit 1658eebed8: Merge pull request #1216 from geoadmin/feat-PB-1031-add-github-link-to-release
Project web-mapviewer
Branch Review develop
Run status status check passed Passed #4388
Run duration 01m 47s
Commit git commit 1658eebed8: Merge pull request #1216 from geoadmin/feat-PB-1031-add-github-link-to-release
Committer Martin Künzi
View all properties for this run ↗︎

Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 0
Tests that did not run due to a developer annotating a test with .skip  Pending 1
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 48
View all changes introduced in this branch ↗︎

@ltkum ltkum force-pushed the feat-PB-1031-add-github-link-to-release branch from 972f675 to d6d0fcf Compare January 21, 2025 15:23
Copy link
Contributor

@pakb pakb left a comment

Choose a reason for hiding this comment

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

I'd separate the AppVersion button into two buttons, one with only the GitHub icon (and that redirect to https://github.com/geoadmin/web-mapviewer) and the other one containing the version (and redirect to the version if and only if the version is stable (not containing dirty or beta keyword)

@ltkum ltkum force-pushed the feat-PB-1031-add-github-link-to-release branch from d966bbc to bbf7931 Compare January 23, 2025 10:19
@ltkum ltkum requested a review from pakb January 23, 2025 10:37
@ltkum ltkum force-pushed the feat-PB-1031-add-github-link-to-release branch from bbf7931 to 7208133 Compare January 23, 2025 13:48
ltkum added 2 commits January 28, 2025 11:37
Github link now leads to the release notes
- The github icon link redirects to the repository
- The app version link redirects to the release IF this is a clean version, otherwise it leads to the repository
@ltkum ltkum force-pushed the feat-PB-1031-add-github-link-to-release branch from 7208133 to 450c91f Compare January 28, 2025 10:37
@ltkum ltkum merged commit 1658eeb into develop Jan 28, 2025
6 checks passed
@ltkum ltkum deleted the feat-PB-1031-add-github-link-to-release branch January 28, 2025 11:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants