-
Notifications
You must be signed in to change notification settings - Fork 237
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
[Chrome-ext] Update nightly download link and add version check #1720
Conversation
a77c6a8
to
f0d6d95
Compare
|
||
export const TelemetryViewerPopup = (): React.ReactElement => { | ||
const [appPhase, setAppPhase] = React.useState<AppPhase>("Startup"); | ||
const [session, setSession] = React.useState<Session | undefined>(undefined); | ||
const [configurationType, setConfigurationType] = React.useState<ConfigurationType>(undefined); | ||
let newAvailableVersion: string; | ||
// let newVersionDownload: string; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Was originally planning on having some sort of dynamic update -- but that is more work so this is really a placeholder
if ((count % 2) == 0) { | ||
document.title = orgTitle; | ||
} else { | ||
document.title = orgTitle + " *** v" + encodeURIComponent(newAvailableVersion) + " Available ***" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Very simple "flashing" title to show the new available version number (really just a different version -- assumes that we only move forward)
@@ -462,7 +462,7 @@ const setPackageJsonRelease = () => { | |||
} | |||
|
|||
if (theFilename.indexOf("/package.json") !== -1) { | |||
let srcFolder = theFilename.replace("/package.json", "/**/*.ts"); | |||
let srcFolder = theFilename.replace("/package.json", "/**/*.ts", "/**/*.tsx", "/**/*.html"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just update our build level version replacement logic to support updating the build versions in *.tsx and *.html files -- this is used for manually updating versions via npm run setVersion -- 2.x.x
and the nightly builds
No description provided.