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

Pm/display version from GitHub #977

Merged
merged 18 commits into from
Jul 11, 2023
Merged

Conversation

PatrikMatiasko
Copy link
Contributor

No description provided.

@@ -5,7 +5,7 @@
<meta charset="utf-8" />
<link rel="icon" href="%PUBLIC_URL%/favicon.png" />
<link href="https://fonts.googleapis.com/css?family=Mulish:400,400i,700" rel="stylesheet">
<meta name="viewport" content="width=device-width, initial-scale=1" />
Copy link
Member

Choose a reason for hiding this comment

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

Do we want to have fixed size to 768 ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ondrej wants app without scrollbars, so I removed this meta tag.

text: _(t.clickHere),
onClick: (e) => {
e.preventDefault()
window.open(GITHUB_VERSION_URL, '_blank')
Copy link
Member

@jkralik jkralik Jul 6, 2023

Choose a reason for hiding this comment

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

use buildInformation.releaseUrl it it set - github URL is not pretty. If empty dont put the link in widget.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

buildInformation.releaseUrl return url for current version but we want to show latest version.

Copy link
Member

@jkralik jkralik Jul 7, 2023

Choose a reason for hiding this comment

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

You are right - you need to read json from GITHUB_VERSION_URL to get property "html_url":

"html_url": "https://github.com/plgd-dev/hub/releases/tag/v2.7.19",

Copy link
Contributor Author

Choose a reason for hiding this comment

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

So, you wanna show github release url, no that json ;)

@@ -25,7 +25,7 @@ const getBuildInformation = (wellKnownConfig: WellKnownConfigType) => ({
commitHash: wellKnownConfig?.commitHash || '',
commitDate: wellKnownConfig?.commitDate || '',
releaseUrl: wellKnownConfig?.releaseUrl || '',
version: wellKnownConfig?.version || '',
version: wellKnownConfig?.version || '0.0.0',
Copy link
Member

Choose a reason for hiding this comment

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

This informations are in hub under object buildInfo:
image

Copy link
Member

Choose a reason for hiding this comment

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

I want to put them the similar way also in client-application.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

So I can use it from buildInfo also in client app ?

Copy link
Member

@jkralik jkralik Jul 7, 2023

Choose a reason for hiding this comment

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

Yes I will move all these properties under buildInfo object:

    buildDate: wellKnownConfig?.buildInfo?.buildDate || '',
    commitHash: wellKnownConfig?.buildInfo?.commitHash || '',
    commitDate: wellKnownConfig?.buildInfo?.commitDate || '',
    releaseUrl: wellKnownConfig?.buildInfo?.releaseUrl || '',
    version: wellKnownConfig?.buildInfo?.version || '0.0.0',

update dependencies
# Conflicts:
#	http-gateway/web/package-lock.json
update dependencies
@sonarcloud
Copy link

sonarcloud bot commented Jul 10, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@PatrikMatiasko PatrikMatiasko merged commit 423cce1 into main Jul 11, 2023
27 checks passed
@Danielius1922 Danielius1922 deleted the pm/display-version-from-github branch November 3, 2023 16:07
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.

2 participants