This repository was archived by the owner on Aug 29, 2025. It is now read-only.

Dash 1.0 - Update version to 1.0.0a1 - #97

Merged
Marc-Andre-Rivet merged 8 commits into
release-v1from
upgrade_react_16.6.1
Dec 18, 2018
Merged

Dash 1.0 - Update version to 1.0.0a1#97
Marc-Andre-Rivet merged 8 commits into
release-v1from
upgrade_react_16.6.1

Conversation

@valentijnnieman

@valentijnniemanvalentijnnieman commented Nov 12, 2018

Copy link
Copy Markdown
Contributor

This PR upgrades React to 16.6.1, and bumps the version up to 1.0.0-rc1.

Relies on:
dash-core-components 1.0.0
dash-html-components 1.0.0
dash 1.0.0

===========================
@Marc-Andre-Rivet is hijacking this existing PR from @valentijnnieman 😄

Fixeshttps://github.com/plotly/dash-core/issues/4

  • Bumps version to 1.0.0a1 (1.0.0-alpha1) instead of rc1 to better respect semver
  • Updating internal references to 16.6.3 (bumping 16.2.0, dropping 15.x)
  • Updating internal copy of react and react-dom min files
  • Updated destination branch to release-v1
  • Not taking care of dev-requirements.txt yet

@nite

nite commented Dec 3, 2018

Copy link
Copy Markdown

Is there any way to install this rc release via pip? Or, anything blocking merging it to master for full release?

@tcbegley

Copy link
Copy Markdown

Hey @nite, you can install with pip install dash-renderer==1.0.0rc1.

I'm not in a position to comment on the plans for merging, but since this introduces some pretty major changes I think it won't happen imminently. Check this issue for some pretty extensive discussion of the upcoming changes.

@Marc-Andre-Rivet
Marc-Andre-Rivet changed the base branch from master to release-v1December 17, 2018 19:01
@Marc-Andre-RivetMarc-Andre-Rivet changed the title Upgrade to React 16.6.1Dash 1.0 - Update version to 1.0.0a1Dec 17, 2018
Marc-André Rivet added 6 commits December 17, 2018 14:12
# Conflicts:
#	CHANGELOG.md
#	dash_renderer/dash_renderer.dev.js
#	dash_renderer/dash_renderer.min.js
#	dash_renderer/version.py
#	package-lock.json
#	package.json
@Marc-Andre-RivetMarc-Andre-Rivet removed the dash 2.0 For issues involving changes potentially slated for Dash 2.0 label Dec 17, 2018
@Marc-Andre-Rivet
Marc-Andre-Rivet removed the request for review from bpostlethwaiteDecember 17, 2018 22:45
@Marc-Andre-Rivet

Marc-Andre-Rivet commented Dec 17, 2018

Copy link
Copy Markdown
Contributor

@nite@tcbegley Thanks for your patience. The dash-renderer and dash-core-components have both previously been pre-released under 1.0.0rc1, this release will be deleted and replaced with 1.0.0a1. We are doing this in an attempt to better match semantic versioning expectations -- an rc release should have a fairly stable implementation and contract with the outside and this react version bump is but the first part of this Dash 1.0 transition, meaning that at this point in time there is no guarantee that the implementation is stable project wide or that we will not significantly change the contract.

As 1.0.0a1 < 1.0.0rc1 this may cause some inconvenience and I apologize for that.

// Look if it was a css file.
let was_css = false;
for (let a of reloadRequest.content.files) {
for (const a of reloadRequest.content.files) {

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.

lint auto fix

Comment threadtests/test_render.py
'external_url': [
'https://unpkg.com/react@15.4.2/dist/react.min.js',
'https://unpkg.com/react-dom@15.4.2/dist/react-dom.min.js',
'https://unpkg.com/react@16.6.3/umd/react.production.min.js',

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.

update our local and remote react versions

_REACT_VERSION_TYPES = {'16.6.3'}
_REACT_VERSION_TO_URLS = {
'15.4.2': {
'16.6.3': {

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.

Bump to 16.x, remove 15.x

Comment threaddev-requirements.txt
dash_core_components==0.33.0
dash_html_components==0.11.0rc5
dash==0.32.0
dash_html_components==0.13.4

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.

There's a fix in html comps for react 16, needs to be present for all tests to pass

@bpostlethwaitebpostlethwaite left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Great!

Going forward how should reviewers ensure / feel confident that these PRs are going to be tested in our internal applications for real world testing?

Is there going to be another PR for release-v1 -> master where we will have a checkbox list after the last commit containing all the apps we need to have tested in?

Comment threaddev-requirements.txt
dash_html_components==0.11.0rc5
dash==0.32.0
dash_html_components==0.13.4
dash==0.32.2

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

should we lock on to rc releases (github branch) here so we are testing against the latest of everything?

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.

We're already planning on making these point to head of master / release-v1 soon

@Marc-Andre-Rivet

Copy link
Copy Markdown
Contributor

@bpostlethwaite A follow up to these is having a release-v1 of the dash-docs. Haven't included a task to test in at least of subset of existing apps when getting ready to push back into master. If you have a good subset in mind, would you mind creating an issue with some ideas on this in the dash-core repo and tagging it with Dash 1.0 label?

@Marc-Andre-Rivet
Marc-Andre-Rivet merged commit 8d1df22 into release-v1Dec 18, 2018
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants

@valentijnnieman@nite@tcbegley@Marc-Andre-Rivet@bpostlethwaite@chriddyp
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Add copy buttons to all
 blocks\n(function() {\n function addCopyButtons() {\n document.querySelectorAll('pre code').forEach(function(codeBlock) {\n if (codeBlock.parentElement.hasAttribute('data-copy-added')) return;\n codeBlock.parentElement.setAttribute('data-copy-added', 'true');\n \n var btn = document.createElement('button');\n btn.textContent = 'Copy';\n btn.style.cssText = 'position:absolute;top:4px;right:4px;padding:2px 8px;font-size:11px;background:#4ecdc4;border:none;border-radius:4px;color:#1a1a2e;cursor:pointer;opacity:0.7;transition:opacity 0.2s;';\n btn.onmouseover = function() { this.style.opacity = '1'; };\n btn.onmouseout = function() { this.style.opacity = '0.7'; };\n btn.onclick = function() {\n navigator.clipboard.writeText(codeBlock.textContent).then(function() {\n btn.textContent = 'Copied!';\n setTimeout(function() { btn.textContent = 'Copy'; }, 1500);\n });\n };\n codeBlock.parentElement.style.position = 'relative';\n codeBlock.parentElement.appendChild(btn);\n });\n }\n \n addCopyButtons();\n \n // Re-run on dynamic content\n var observer = new MutationObserver(addCopyButtons);\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Add Copy Buttons to Code Blocks");
}
} catch(__e) { console.warn('[Userscript:Add Copy Buttons to Code Blocks]', __e); }
})();
(function(){
try {
var __m = "github.com";
var __re = new RegExp('^' + "github\\.com" + '
Skip to content
This repository was archived by the owner on Aug 29, 2025. It is now read-only.

Dash 1.0 - Update version to 1.0.0a1 - #97

Merged
Marc-Andre-Rivet merged 8 commits into
release-v1from
upgrade_react_16.6.1
Dec 18, 2018
Merged

Dash 1.0 - Update version to 1.0.0a1#97
Marc-Andre-Rivet merged 8 commits into
release-v1from
upgrade_react_16.6.1

Conversation

@valentijnnieman

@valentijnniemanvalentijnnieman commented Nov 12, 2018

Copy link
Copy Markdown
Contributor

This PR upgrades React to 16.6.1, and bumps the version up to 1.0.0-rc1.

Relies on:
dash-core-components 1.0.0
dash-html-components 1.0.0
dash 1.0.0

===========================
@Marc-Andre-Rivet is hijacking this existing PR from @valentijnnieman 😄

Fixeshttps://github.com/plotly/dash-core/issues/4

  • Bumps version to 1.0.0a1 (1.0.0-alpha1) instead of rc1 to better respect semver
  • Updating internal references to 16.6.3 (bumping 16.2.0, dropping 15.x)
  • Updating internal copy of react and react-dom min files
  • Updated destination branch to release-v1
  • Not taking care of dev-requirements.txt yet

@nite

nite commented Dec 3, 2018

Copy link
Copy Markdown

Is there any way to install this rc release via pip? Or, anything blocking merging it to master for full release?

@tcbegley

Copy link
Copy Markdown

Hey @nite, you can install with pip install dash-renderer==1.0.0rc1.

I'm not in a position to comment on the plans for merging, but since this introduces some pretty major changes I think it won't happen imminently. Check this issue for some pretty extensive discussion of the upcoming changes.

@Marc-Andre-Rivet
Marc-Andre-Rivet changed the base branch from master to release-v1December 17, 2018 19:01
@Marc-Andre-RivetMarc-Andre-Rivet changed the title Upgrade to React 16.6.1Dash 1.0 - Update version to 1.0.0a1Dec 17, 2018
Marc-André Rivet added 6 commits December 17, 2018 14:12
# Conflicts:
#	CHANGELOG.md
#	dash_renderer/dash_renderer.dev.js
#	dash_renderer/dash_renderer.min.js
#	dash_renderer/version.py
#	package-lock.json
#	package.json
@Marc-Andre-RivetMarc-Andre-Rivet removed the dash 2.0 For issues involving changes potentially slated for Dash 2.0 label Dec 17, 2018
@Marc-Andre-Rivet
Marc-Andre-Rivet removed the request for review from bpostlethwaiteDecember 17, 2018 22:45
@Marc-Andre-Rivet

Marc-Andre-Rivet commented Dec 17, 2018

Copy link
Copy Markdown
Contributor

@nite@tcbegley Thanks for your patience. The dash-renderer and dash-core-components have both previously been pre-released under 1.0.0rc1, this release will be deleted and replaced with 1.0.0a1. We are doing this in an attempt to better match semantic versioning expectations -- an rc release should have a fairly stable implementation and contract with the outside and this react version bump is but the first part of this Dash 1.0 transition, meaning that at this point in time there is no guarantee that the implementation is stable project wide or that we will not significantly change the contract.

As 1.0.0a1 < 1.0.0rc1 this may cause some inconvenience and I apologize for that.

// Look if it was a css file.
let was_css = false;
for (let a of reloadRequest.content.files) {
for (const a of reloadRequest.content.files) {

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.

lint auto fix

Comment threadtests/test_render.py
'external_url': [
'https://unpkg.com/react@15.4.2/dist/react.min.js',
'https://unpkg.com/react-dom@15.4.2/dist/react-dom.min.js',
'https://unpkg.com/react@16.6.3/umd/react.production.min.js',

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.

update our local and remote react versions

_REACT_VERSION_TYPES = {'16.6.3'}
_REACT_VERSION_TO_URLS = {
'15.4.2': {
'16.6.3': {

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.

Bump to 16.x, remove 15.x

Comment threaddev-requirements.txt
dash_core_components==0.33.0
dash_html_components==0.11.0rc5
dash==0.32.0
dash_html_components==0.13.4

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.

There's a fix in html comps for react 16, needs to be present for all tests to pass

@bpostlethwaitebpostlethwaite left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Great!

Going forward how should reviewers ensure / feel confident that these PRs are going to be tested in our internal applications for real world testing?

Is there going to be another PR for release-v1 -> master where we will have a checkbox list after the last commit containing all the apps we need to have tested in?

Comment threaddev-requirements.txt
dash_html_components==0.11.0rc5
dash==0.32.0
dash_html_components==0.13.4
dash==0.32.2

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

should we lock on to rc releases (github branch) here so we are testing against the latest of everything?

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.

We're already planning on making these point to head of master / release-v1 soon

@Marc-Andre-Rivet

Copy link
Copy Markdown
Contributor

@bpostlethwaite A follow up to these is having a release-v1 of the dash-docs. Haven't included a task to test in at least of subset of existing apps when getting ready to push back into master. If you have a good subset in mind, would you mind creating an issue with some ideas on this in the dash-core repo and tagging it with Dash 1.0 label?

@Marc-Andre-Rivet
Marc-Andre-Rivet merged commit 8d1df22 into release-v1Dec 18, 2018
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants

@valentijnnieman@nite@tcbegley@Marc-Andre-Rivet@bpostlethwaite@chriddyp
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Force GitHub README to respect dark mode\n(function() {\n var style = document.createElement('style');\n style.textContent = '\n .markdown-body {\n color-scheme: dark light;\n }\n .markdown-body pre { background: #161b22 !important; }\n .markdown-body code { background: rgba(110, 118, 129, 0.4) !important; }\n .markdown-body table th, .markdown-body table td { border-color: #30363d !important; }\n .markdown-body img { background: #0d1117; }\n .markdown-body blockquote { border-left-color: #8b949e; }\n .markdown-body hr { border-color: #30363d; }\n ';\n document.head.appendChild(style);\n})();", "GitHub Dark Mode README Fix"); } } catch(__e) { console.warn('[Userscript:GitHub Dark Mode README Fix]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + '
Skip to content
This repository was archived by the owner on Aug 29, 2025. It is now read-only.

Dash 1.0 - Update version to 1.0.0a1 - #97

Merged
Marc-Andre-Rivet merged 8 commits into
release-v1from
upgrade_react_16.6.1
Dec 18, 2018
Merged

Dash 1.0 - Update version to 1.0.0a1#97
Marc-Andre-Rivet merged 8 commits into
release-v1from
upgrade_react_16.6.1

Conversation

@valentijnnieman

@valentijnniemanvalentijnnieman commented Nov 12, 2018

Copy link
Copy Markdown
Contributor

This PR upgrades React to 16.6.1, and bumps the version up to 1.0.0-rc1.

Relies on:
dash-core-components 1.0.0
dash-html-components 1.0.0
dash 1.0.0

===========================
@Marc-Andre-Rivet is hijacking this existing PR from @valentijnnieman 😄

Fixeshttps://github.com/plotly/dash-core/issues/4

  • Bumps version to 1.0.0a1 (1.0.0-alpha1) instead of rc1 to better respect semver
  • Updating internal references to 16.6.3 (bumping 16.2.0, dropping 15.x)
  • Updating internal copy of react and react-dom min files
  • Updated destination branch to release-v1
  • Not taking care of dev-requirements.txt yet

@nite

nite commented Dec 3, 2018

Copy link
Copy Markdown

Is there any way to install this rc release via pip? Or, anything blocking merging it to master for full release?

@tcbegley

Copy link
Copy Markdown

Hey @nite, you can install with pip install dash-renderer==1.0.0rc1.

I'm not in a position to comment on the plans for merging, but since this introduces some pretty major changes I think it won't happen imminently. Check this issue for some pretty extensive discussion of the upcoming changes.

@Marc-Andre-Rivet
Marc-Andre-Rivet changed the base branch from master to release-v1December 17, 2018 19:01
@Marc-Andre-RivetMarc-Andre-Rivet changed the title Upgrade to React 16.6.1Dash 1.0 - Update version to 1.0.0a1Dec 17, 2018
Marc-André Rivet added 6 commits December 17, 2018 14:12
# Conflicts:
#	CHANGELOG.md
#	dash_renderer/dash_renderer.dev.js
#	dash_renderer/dash_renderer.min.js
#	dash_renderer/version.py
#	package-lock.json
#	package.json
@Marc-Andre-RivetMarc-Andre-Rivet removed the dash 2.0 For issues involving changes potentially slated for Dash 2.0 label Dec 17, 2018
@Marc-Andre-Rivet
Marc-Andre-Rivet removed the request for review from bpostlethwaiteDecember 17, 2018 22:45
@Marc-Andre-Rivet

Marc-Andre-Rivet commented Dec 17, 2018

Copy link
Copy Markdown
Contributor

@nite@tcbegley Thanks for your patience. The dash-renderer and dash-core-components have both previously been pre-released under 1.0.0rc1, this release will be deleted and replaced with 1.0.0a1. We are doing this in an attempt to better match semantic versioning expectations -- an rc release should have a fairly stable implementation and contract with the outside and this react version bump is but the first part of this Dash 1.0 transition, meaning that at this point in time there is no guarantee that the implementation is stable project wide or that we will not significantly change the contract.

As 1.0.0a1 < 1.0.0rc1 this may cause some inconvenience and I apologize for that.

// Look if it was a css file.
let was_css = false;
for (let a of reloadRequest.content.files) {
for (const a of reloadRequest.content.files) {

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.

lint auto fix

Comment threadtests/test_render.py
'external_url': [
'https://unpkg.com/react@15.4.2/dist/react.min.js',
'https://unpkg.com/react-dom@15.4.2/dist/react-dom.min.js',
'https://unpkg.com/react@16.6.3/umd/react.production.min.js',

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.

update our local and remote react versions

_REACT_VERSION_TYPES = {'16.6.3'}
_REACT_VERSION_TO_URLS = {
'15.4.2': {
'16.6.3': {

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.

Bump to 16.x, remove 15.x

Comment threaddev-requirements.txt
dash_core_components==0.33.0
dash_html_components==0.11.0rc5
dash==0.32.0
dash_html_components==0.13.4

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.

There's a fix in html comps for react 16, needs to be present for all tests to pass

@bpostlethwaitebpostlethwaite left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Great!

Going forward how should reviewers ensure / feel confident that these PRs are going to be tested in our internal applications for real world testing?

Is there going to be another PR for release-v1 -> master where we will have a checkbox list after the last commit containing all the apps we need to have tested in?

Comment threaddev-requirements.txt
dash_html_components==0.11.0rc5
dash==0.32.0
dash_html_components==0.13.4
dash==0.32.2

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

should we lock on to rc releases (github branch) here so we are testing against the latest of everything?

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.

We're already planning on making these point to head of master / release-v1 soon

@Marc-Andre-Rivet

Copy link
Copy Markdown
Contributor

@bpostlethwaite A follow up to these is having a release-v1 of the dash-docs. Haven't included a task to test in at least of subset of existing apps when getting ready to push back into master. If you have a good subset in mind, would you mind creating an issue with some ideas on this in the dash-core repo and tagging it with Dash 1.0 label?

@Marc-Andre-Rivet
Marc-Andre-Rivet merged commit 8d1df22 into release-v1Dec 18, 2018
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants

@valentijnnieman@nite@tcbegley@Marc-Andre-Rivet@bpostlethwaite@chriddyp
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Highlight search terms from Google/DuckDuckGo/Bing referrer\n(function() {\n var ref = document.referrer;\n var terms = [];\n \n if (ref.includes('google.com') || ref.includes('duckduckgo.com') || ref.includes('bing.com')) {\n var url = new URL(ref);\n var q = url.searchParams.get('q') || url.searchParams.get('p');\n if (q) {\n terms = q.split(/\\s+/).filter(function(t) { return t.length > 2; });\n }\n }\n \n if (terms.length === 0) return;\n \n var style = document.createElement('style');\n style.textContent = '.userscript-highlight { background: #fbbf24; color: #1a1a2e; padding: 1px 3px; border-radius: 2px; }';\n document.head.appendChild(style);\n \n function highlight(node) {\n if (node.nodeType === 3) { // text node\n var text = node.textContent;\n var found = false;\n terms.forEach(function(term) {\n var regex = new RegExp('(' + term.replace(/[.*+?^${}()|[\\]\\\\]/g, '\\\\') + ')', 'gi');\n if (regex.test(text)) {\n found = true;\n var frag = document.createDocumentFragment();\n var parts = text.split(regex);\n parts.forEach(function(part, i) {\n if (i % 2 === 0) {\n frag.appendChild(document.createTextNode(part));\n } else {\n var span = document.createElement('span');\n span.className = 'userscript-highlight';\n span.textContent = part;\n frag.appendChild(span);\n }\n });\n node.parentNode.replaceChild(frag, node);\n }\n });\n } else if (node.nodeType === 1 && node.childNodes) { // element\n var skipTags = ['SCRIPT', 'STYLE', 'NOSCRIPT', 'TEXTAREA', 'INPUT', 'SELECT'];\n if (!skipTags.includes(node.tagName)) {\n Array.from(node.childNodes).forEach(highlight);\n }\n }\n }\n \n highlight(document.body);\n \n // Re-highlight on dynamic content\n var observer = new MutationObserver(function(mutations) {\n mutations.forEach(function(m) {\n m.addedNodes.forEach(function(node) {\n if (node.nodeType === 1 || node.nodeType === 3) highlight(node);\n });\n });\n });\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Highlight Search Terms"); } } catch(__e) { console.warn('[Userscript:Highlight Search Terms]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + '
Skip to content
This repository was archived by the owner on Aug 29, 2025. It is now read-only.

Dash 1.0 - Update version to 1.0.0a1 - #97

Merged
Marc-Andre-Rivet merged 8 commits into
release-v1from
upgrade_react_16.6.1
Dec 18, 2018
Merged

Dash 1.0 - Update version to 1.0.0a1#97
Marc-Andre-Rivet merged 8 commits into
release-v1from
upgrade_react_16.6.1

Conversation

@valentijnnieman

@valentijnniemanvalentijnnieman commented Nov 12, 2018

Copy link
Copy Markdown
Contributor

This PR upgrades React to 16.6.1, and bumps the version up to 1.0.0-rc1.

Relies on:
dash-core-components 1.0.0
dash-html-components 1.0.0
dash 1.0.0

===========================
@Marc-Andre-Rivet is hijacking this existing PR from @valentijnnieman 😄

Fixeshttps://github.com/plotly/dash-core/issues/4

  • Bumps version to 1.0.0a1 (1.0.0-alpha1) instead of rc1 to better respect semver
  • Updating internal references to 16.6.3 (bumping 16.2.0, dropping 15.x)
  • Updating internal copy of react and react-dom min files
  • Updated destination branch to release-v1
  • Not taking care of dev-requirements.txt yet

@nite

nite commented Dec 3, 2018

Copy link
Copy Markdown

Is there any way to install this rc release via pip? Or, anything blocking merging it to master for full release?

@tcbegley

Copy link
Copy Markdown

Hey @nite, you can install with pip install dash-renderer==1.0.0rc1.

I'm not in a position to comment on the plans for merging, but since this introduces some pretty major changes I think it won't happen imminently. Check this issue for some pretty extensive discussion of the upcoming changes.

@Marc-Andre-Rivet
Marc-Andre-Rivet changed the base branch from master to release-v1December 17, 2018 19:01
@Marc-Andre-RivetMarc-Andre-Rivet changed the title Upgrade to React 16.6.1Dash 1.0 - Update version to 1.0.0a1Dec 17, 2018
Marc-André Rivet added 6 commits December 17, 2018 14:12
# Conflicts:
#	CHANGELOG.md
#	dash_renderer/dash_renderer.dev.js
#	dash_renderer/dash_renderer.min.js
#	dash_renderer/version.py
#	package-lock.json
#	package.json
@Marc-Andre-RivetMarc-Andre-Rivet removed the dash 2.0 For issues involving changes potentially slated for Dash 2.0 label Dec 17, 2018
@Marc-Andre-Rivet
Marc-Andre-Rivet removed the request for review from bpostlethwaiteDecember 17, 2018 22:45
@Marc-Andre-Rivet

Marc-Andre-Rivet commented Dec 17, 2018

Copy link
Copy Markdown
Contributor

@nite@tcbegley Thanks for your patience. The dash-renderer and dash-core-components have both previously been pre-released under 1.0.0rc1, this release will be deleted and replaced with 1.0.0a1. We are doing this in an attempt to better match semantic versioning expectations -- an rc release should have a fairly stable implementation and contract with the outside and this react version bump is but the first part of this Dash 1.0 transition, meaning that at this point in time there is no guarantee that the implementation is stable project wide or that we will not significantly change the contract.

As 1.0.0a1 < 1.0.0rc1 this may cause some inconvenience and I apologize for that.

// Look if it was a css file.
let was_css = false;
for (let a of reloadRequest.content.files) {
for (const a of reloadRequest.content.files) {

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.

lint auto fix

Comment threadtests/test_render.py
'external_url': [
'https://unpkg.com/react@15.4.2/dist/react.min.js',
'https://unpkg.com/react-dom@15.4.2/dist/react-dom.min.js',
'https://unpkg.com/react@16.6.3/umd/react.production.min.js',

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.

update our local and remote react versions

_REACT_VERSION_TYPES = {'16.6.3'}
_REACT_VERSION_TO_URLS = {
'15.4.2': {
'16.6.3': {

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.

Bump to 16.x, remove 15.x

Comment threaddev-requirements.txt
dash_core_components==0.33.0
dash_html_components==0.11.0rc5
dash==0.32.0
dash_html_components==0.13.4

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.

There's a fix in html comps for react 16, needs to be present for all tests to pass

@bpostlethwaitebpostlethwaite left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Great!

Going forward how should reviewers ensure / feel confident that these PRs are going to be tested in our internal applications for real world testing?

Is there going to be another PR for release-v1 -> master where we will have a checkbox list after the last commit containing all the apps we need to have tested in?

Comment threaddev-requirements.txt
dash_html_components==0.11.0rc5
dash==0.32.0
dash_html_components==0.13.4
dash==0.32.2

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

should we lock on to rc releases (github branch) here so we are testing against the latest of everything?

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.

We're already planning on making these point to head of master / release-v1 soon

@Marc-Andre-Rivet

Copy link
Copy Markdown
Contributor

@bpostlethwaite A follow up to these is having a release-v1 of the dash-docs. Haven't included a task to test in at least of subset of existing apps when getting ready to push back into master. If you have a good subset in mind, would you mind creating an issue with some ideas on this in the dash-core repo and tagging it with Dash 1.0 label?

@Marc-Andre-Rivet
Marc-Andre-Rivet merged commit 8d1df22 into release-v1Dec 18, 2018
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants

@valentijnnieman@nite@tcbegley@Marc-Andre-Rivet@bpostlethwaite@chriddyp
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Strip utm_, fbclid, gclid, etc. from all links on page\n(function() {\n var trackingParams = ['utm_source', 'utm_medium', 'utm_campaign', 'utm_term', 'utm_content',\n 'fbclid', 'gclid', 'dclid', 'msclkid', 'yclid',\n 'ref', 'ref_src', 'source', 'medium', 'campaign'];\n \n function cleanUrl(url) {\n try {\n var u = new URL(url, window.location.origin);\n var changed = false;\n trackingParams.forEach(function(p) {\n if (u.searchParams.has(p)) {\n u.searchParams.delete(p);\n changed = true;\n }\n });\n return changed ? u.toString() : url;\n } catch (e) {\n return url;\n }\n }\n \n function cleanLinks() {\n document.querySelectorAll('a[href]').forEach(function(a) {\n var clean = cleanUrl(a.href);\n if (clean !== a.href) a.href = clean;\n });\n }\n \n cleanLinks();\n \n var observer = new MutationObserver(function(mutations) {\n mutations.forEach(function(m) {\n m.addedNodes.forEach(function(node) {\n if (node.nodeType === 1) {\n if (node.tagName === 'A') cleanLinks();\n node.querySelectorAll('a[href]').forEach(function(a) {\n var clean = cleanUrl(a.href);\n if (clean !== a.href) a.href = clean;\n });\n }\n });\n });\n });\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Remove Tracking Parameters from Links"); } } catch(__e) { console.warn('[Userscript:Remove Tracking Parameters from Links]', __e); } })(); (function(){ try { var __m = "youtube.com"; var __re = new RegExp('^' + "youtube\\.com" + '
Skip to content
This repository was archived by the owner on Aug 29, 2025. It is now read-only.

Dash 1.0 - Update version to 1.0.0a1 - #97

Merged
Marc-Andre-Rivet merged 8 commits into
release-v1from
upgrade_react_16.6.1
Dec 18, 2018
Merged

Dash 1.0 - Update version to 1.0.0a1#97
Marc-Andre-Rivet merged 8 commits into
release-v1from
upgrade_react_16.6.1

Conversation

@valentijnnieman

@valentijnniemanvalentijnnieman commented Nov 12, 2018

Copy link
Copy Markdown
Contributor

This PR upgrades React to 16.6.1, and bumps the version up to 1.0.0-rc1.

Relies on:
dash-core-components 1.0.0
dash-html-components 1.0.0
dash 1.0.0

===========================
@Marc-Andre-Rivet is hijacking this existing PR from @valentijnnieman 😄

Fixeshttps://github.com/plotly/dash-core/issues/4

  • Bumps version to 1.0.0a1 (1.0.0-alpha1) instead of rc1 to better respect semver
  • Updating internal references to 16.6.3 (bumping 16.2.0, dropping 15.x)
  • Updating internal copy of react and react-dom min files
  • Updated destination branch to release-v1
  • Not taking care of dev-requirements.txt yet

@nite

nite commented Dec 3, 2018

Copy link
Copy Markdown

Is there any way to install this rc release via pip? Or, anything blocking merging it to master for full release?

@tcbegley

Copy link
Copy Markdown

Hey @nite, you can install with pip install dash-renderer==1.0.0rc1.

I'm not in a position to comment on the plans for merging, but since this introduces some pretty major changes I think it won't happen imminently. Check this issue for some pretty extensive discussion of the upcoming changes.

@Marc-Andre-Rivet
Marc-Andre-Rivet changed the base branch from master to release-v1December 17, 2018 19:01
@Marc-Andre-RivetMarc-Andre-Rivet changed the title Upgrade to React 16.6.1Dash 1.0 - Update version to 1.0.0a1Dec 17, 2018
Marc-André Rivet added 6 commits December 17, 2018 14:12
# Conflicts:
#	CHANGELOG.md
#	dash_renderer/dash_renderer.dev.js
#	dash_renderer/dash_renderer.min.js
#	dash_renderer/version.py
#	package-lock.json
#	package.json
@Marc-Andre-RivetMarc-Andre-Rivet removed the dash 2.0 For issues involving changes potentially slated for Dash 2.0 label Dec 17, 2018
@Marc-Andre-Rivet
Marc-Andre-Rivet removed the request for review from bpostlethwaiteDecember 17, 2018 22:45
@Marc-Andre-Rivet

Marc-Andre-Rivet commented Dec 17, 2018

Copy link
Copy Markdown
Contributor

@nite@tcbegley Thanks for your patience. The dash-renderer and dash-core-components have both previously been pre-released under 1.0.0rc1, this release will be deleted and replaced with 1.0.0a1. We are doing this in an attempt to better match semantic versioning expectations -- an rc release should have a fairly stable implementation and contract with the outside and this react version bump is but the first part of this Dash 1.0 transition, meaning that at this point in time there is no guarantee that the implementation is stable project wide or that we will not significantly change the contract.

As 1.0.0a1 < 1.0.0rc1 this may cause some inconvenience and I apologize for that.

// Look if it was a css file.
let was_css = false;
for (let a of reloadRequest.content.files) {
for (const a of reloadRequest.content.files) {

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.

lint auto fix

Comment threadtests/test_render.py
'external_url': [
'https://unpkg.com/react@15.4.2/dist/react.min.js',
'https://unpkg.com/react-dom@15.4.2/dist/react-dom.min.js',
'https://unpkg.com/react@16.6.3/umd/react.production.min.js',

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.

update our local and remote react versions

_REACT_VERSION_TYPES = {'16.6.3'}
_REACT_VERSION_TO_URLS = {
'15.4.2': {
'16.6.3': {

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.

Bump to 16.x, remove 15.x

Comment threaddev-requirements.txt
dash_core_components==0.33.0
dash_html_components==0.11.0rc5
dash==0.32.0
dash_html_components==0.13.4

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.

There's a fix in html comps for react 16, needs to be present for all tests to pass

@bpostlethwaitebpostlethwaite left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Great!

Going forward how should reviewers ensure / feel confident that these PRs are going to be tested in our internal applications for real world testing?

Is there going to be another PR for release-v1 -> master where we will have a checkbox list after the last commit containing all the apps we need to have tested in?

Comment threaddev-requirements.txt
dash_html_components==0.11.0rc5
dash==0.32.0
dash_html_components==0.13.4
dash==0.32.2

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

should we lock on to rc releases (github branch) here so we are testing against the latest of everything?

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.

We're already planning on making these point to head of master / release-v1 soon

@Marc-Andre-Rivet

Copy link
Copy Markdown
Contributor

@bpostlethwaite A follow up to these is having a release-v1 of the dash-docs. Haven't included a task to test in at least of subset of existing apps when getting ready to push back into master. If you have a good subset in mind, would you mind creating an issue with some ideas on this in the dash-core repo and tagging it with Dash 1.0 label?

@Marc-Andre-Rivet
Marc-Andre-Rivet merged commit 8d1df22 into release-v1Dec 18, 2018
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants

@valentijnnieman@nite@tcbegley@Marc-Andre-Rivet@bpostlethwaite@chriddyp
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Auto-enable theater mode on YouTube\n(function() {\n function tryTheater() {\n var btn = document.querySelector('button[aria-label=\"Theater mode\"], ytd-player #player button[title=\"Theater mode\"]');\n if (btn && !btn.classList.contains('activated')) {\n btn.click();\n }\n }\n \n // Try immediately\n tryTheater();\n \n // Try after navigation (SPA)\n var lastUrl = location.href;\n setInterval(function() {\n if (location.href !== lastUrl) {\n lastUrl = location.href;\n setTimeout(tryTheater, 500);\n }\n }, 1000);\n \n // Also try on player load\n var observer = new MutationObserver(tryTheater);\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "YouTube Theater Mode Default"); } } catch(__e) { console.warn('[Userscript:YouTube Theater Mode Default]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + '
Skip to content
This repository was archived by the owner on Aug 29, 2025. It is now read-only.

Dash 1.0 - Update version to 1.0.0a1 - #97

Merged
Marc-Andre-Rivet merged 8 commits into
release-v1from
upgrade_react_16.6.1
Dec 18, 2018
Merged

Dash 1.0 - Update version to 1.0.0a1#97
Marc-Andre-Rivet merged 8 commits into
release-v1from
upgrade_react_16.6.1

Conversation

@valentijnnieman

@valentijnniemanvalentijnnieman commented Nov 12, 2018

Copy link
Copy Markdown
Contributor

This PR upgrades React to 16.6.1, and bumps the version up to 1.0.0-rc1.

Relies on:
dash-core-components 1.0.0
dash-html-components 1.0.0
dash 1.0.0

===========================
@Marc-Andre-Rivet is hijacking this existing PR from @valentijnnieman 😄

Fixeshttps://github.com/plotly/dash-core/issues/4

  • Bumps version to 1.0.0a1 (1.0.0-alpha1) instead of rc1 to better respect semver
  • Updating internal references to 16.6.3 (bumping 16.2.0, dropping 15.x)
  • Updating internal copy of react and react-dom min files
  • Updated destination branch to release-v1
  • Not taking care of dev-requirements.txt yet

@nite

nite commented Dec 3, 2018

Copy link
Copy Markdown

Is there any way to install this rc release via pip? Or, anything blocking merging it to master for full release?

@tcbegley

Copy link
Copy Markdown

Hey @nite, you can install with pip install dash-renderer==1.0.0rc1.

I'm not in a position to comment on the plans for merging, but since this introduces some pretty major changes I think it won't happen imminently. Check this issue for some pretty extensive discussion of the upcoming changes.

@Marc-Andre-Rivet
Marc-Andre-Rivet changed the base branch from master to release-v1December 17, 2018 19:01
@Marc-Andre-RivetMarc-Andre-Rivet changed the title Upgrade to React 16.6.1Dash 1.0 - Update version to 1.0.0a1Dec 17, 2018
Marc-André Rivet added 6 commits December 17, 2018 14:12
# Conflicts:
#	CHANGELOG.md
#	dash_renderer/dash_renderer.dev.js
#	dash_renderer/dash_renderer.min.js
#	dash_renderer/version.py
#	package-lock.json
#	package.json
@Marc-Andre-RivetMarc-Andre-Rivet removed the dash 2.0 For issues involving changes potentially slated for Dash 2.0 label Dec 17, 2018
@Marc-Andre-Rivet
Marc-Andre-Rivet removed the request for review from bpostlethwaiteDecember 17, 2018 22:45
@Marc-Andre-Rivet

Marc-Andre-Rivet commented Dec 17, 2018

Copy link
Copy Markdown
Contributor

@nite@tcbegley Thanks for your patience. The dash-renderer and dash-core-components have both previously been pre-released under 1.0.0rc1, this release will be deleted and replaced with 1.0.0a1. We are doing this in an attempt to better match semantic versioning expectations -- an rc release should have a fairly stable implementation and contract with the outside and this react version bump is but the first part of this Dash 1.0 transition, meaning that at this point in time there is no guarantee that the implementation is stable project wide or that we will not significantly change the contract.

As 1.0.0a1 < 1.0.0rc1 this may cause some inconvenience and I apologize for that.

// Look if it was a css file.
let was_css = false;
for (let a of reloadRequest.content.files) {
for (const a of reloadRequest.content.files) {

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.

lint auto fix

Comment threadtests/test_render.py
'external_url': [
'https://unpkg.com/react@15.4.2/dist/react.min.js',
'https://unpkg.com/react-dom@15.4.2/dist/react-dom.min.js',
'https://unpkg.com/react@16.6.3/umd/react.production.min.js',

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.

update our local and remote react versions

_REACT_VERSION_TYPES = {'16.6.3'}
_REACT_VERSION_TO_URLS = {
'15.4.2': {
'16.6.3': {

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.

Bump to 16.x, remove 15.x

Comment threaddev-requirements.txt
dash_core_components==0.33.0
dash_html_components==0.11.0rc5
dash==0.32.0
dash_html_components==0.13.4

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.

There's a fix in html comps for react 16, needs to be present for all tests to pass

@bpostlethwaitebpostlethwaite left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Great!

Going forward how should reviewers ensure / feel confident that these PRs are going to be tested in our internal applications for real world testing?

Is there going to be another PR for release-v1 -> master where we will have a checkbox list after the last commit containing all the apps we need to have tested in?

Comment threaddev-requirements.txt
dash_html_components==0.11.0rc5
dash==0.32.0
dash_html_components==0.13.4
dash==0.32.2

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

should we lock on to rc releases (github branch) here so we are testing against the latest of everything?

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.

We're already planning on making these point to head of master / release-v1 soon

@Marc-Andre-Rivet

Copy link
Copy Markdown
Contributor

@bpostlethwaite A follow up to these is having a release-v1 of the dash-docs. Haven't included a task to test in at least of subset of existing apps when getting ready to push back into master. If you have a good subset in mind, would you mind creating an issue with some ideas on this in the dash-core repo and tagging it with Dash 1.0 label?

@Marc-Andre-Rivet
Marc-Andre-Rivet merged commit 8d1df22 into release-v1Dec 18, 2018
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants

@valentijnnieman@nite@tcbegley@Marc-Andre-Rivet@bpostlethwaite@chriddyp
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Remove or un-stick sticky/fixed headers that block content\n(function() {\n function unstick() {\n document.querySelectorAll('header, nav, [role=\"banner\"], .header, .navbar, .sticky, .fixed-top, [style*=\"position: fixed\"], [style*=\"position:sticky\"]').forEach(function(el) {\n if (el.style.position === 'fixed' || el.style.position === 'sticky' || \n getComputedStyle(el).position === 'fixed' || getComputedStyle(el).position === 'sticky') {\n el.style.position = 'static';\n el.style.top = 'auto';\n el.style.zIndex = 'auto';\n }\n });\n }\n \n unstick();\n \n var observer = new MutationObserver(unstick);\n observer.observe(document.body, { childList: true, subtree: true, attributes: true, attributeFilter: ['style', 'class'] });\n})();", "Kill Sticky Headers"); } } catch(__e) { console.warn('[Userscript:Kill Sticky Headers]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + '
Skip to content
This repository was archived by the owner on Aug 29, 2025. It is now read-only.

Dash 1.0 - Update version to 1.0.0a1 - #97

Merged
Marc-Andre-Rivet merged 8 commits into
release-v1from
upgrade_react_16.6.1
Dec 18, 2018
Merged

Dash 1.0 - Update version to 1.0.0a1#97
Marc-Andre-Rivet merged 8 commits into
release-v1from
upgrade_react_16.6.1

Conversation

@valentijnnieman

@valentijnniemanvalentijnnieman commented Nov 12, 2018

Copy link
Copy Markdown
Contributor

This PR upgrades React to 16.6.1, and bumps the version up to 1.0.0-rc1.

Relies on:
dash-core-components 1.0.0
dash-html-components 1.0.0
dash 1.0.0

===========================
@Marc-Andre-Rivet is hijacking this existing PR from @valentijnnieman 😄

Fixeshttps://github.com/plotly/dash-core/issues/4

  • Bumps version to 1.0.0a1 (1.0.0-alpha1) instead of rc1 to better respect semver
  • Updating internal references to 16.6.3 (bumping 16.2.0, dropping 15.x)
  • Updating internal copy of react and react-dom min files
  • Updated destination branch to release-v1
  • Not taking care of dev-requirements.txt yet

@nite

nite commented Dec 3, 2018

Copy link
Copy Markdown

Is there any way to install this rc release via pip? Or, anything blocking merging it to master for full release?

@tcbegley

Copy link
Copy Markdown

Hey @nite, you can install with pip install dash-renderer==1.0.0rc1.

I'm not in a position to comment on the plans for merging, but since this introduces some pretty major changes I think it won't happen imminently. Check this issue for some pretty extensive discussion of the upcoming changes.

@Marc-Andre-Rivet
Marc-Andre-Rivet changed the base branch from master to release-v1December 17, 2018 19:01
@Marc-Andre-RivetMarc-Andre-Rivet changed the title Upgrade to React 16.6.1Dash 1.0 - Update version to 1.0.0a1Dec 17, 2018
Marc-André Rivet added 6 commits December 17, 2018 14:12
# Conflicts:
#	CHANGELOG.md
#	dash_renderer/dash_renderer.dev.js
#	dash_renderer/dash_renderer.min.js
#	dash_renderer/version.py
#	package-lock.json
#	package.json
@Marc-Andre-RivetMarc-Andre-Rivet removed the dash 2.0 For issues involving changes potentially slated for Dash 2.0 label Dec 17, 2018
@Marc-Andre-Rivet
Marc-Andre-Rivet removed the request for review from bpostlethwaiteDecember 17, 2018 22:45
@Marc-Andre-Rivet

Marc-Andre-Rivet commented Dec 17, 2018

Copy link
Copy Markdown
Contributor

@nite@tcbegley Thanks for your patience. The dash-renderer and dash-core-components have both previously been pre-released under 1.0.0rc1, this release will be deleted and replaced with 1.0.0a1. We are doing this in an attempt to better match semantic versioning expectations -- an rc release should have a fairly stable implementation and contract with the outside and this react version bump is but the first part of this Dash 1.0 transition, meaning that at this point in time there is no guarantee that the implementation is stable project wide or that we will not significantly change the contract.

As 1.0.0a1 < 1.0.0rc1 this may cause some inconvenience and I apologize for that.

// Look if it was a css file.
let was_css = false;
for (let a of reloadRequest.content.files) {
for (const a of reloadRequest.content.files) {

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.

lint auto fix

Comment threadtests/test_render.py
'external_url': [
'https://unpkg.com/react@15.4.2/dist/react.min.js',
'https://unpkg.com/react-dom@15.4.2/dist/react-dom.min.js',
'https://unpkg.com/react@16.6.3/umd/react.production.min.js',

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.

update our local and remote react versions

_REACT_VERSION_TYPES = {'16.6.3'}
_REACT_VERSION_TO_URLS = {
'15.4.2': {
'16.6.3': {

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.

Bump to 16.x, remove 15.x

Comment threaddev-requirements.txt
dash_core_components==0.33.0
dash_html_components==0.11.0rc5
dash==0.32.0
dash_html_components==0.13.4

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.

There's a fix in html comps for react 16, needs to be present for all tests to pass

@bpostlethwaitebpostlethwaite left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Great!

Going forward how should reviewers ensure / feel confident that these PRs are going to be tested in our internal applications for real world testing?

Is there going to be another PR for release-v1 -> master where we will have a checkbox list after the last commit containing all the apps we need to have tested in?

Comment threaddev-requirements.txt
dash_html_components==0.11.0rc5
dash==0.32.0
dash_html_components==0.13.4
dash==0.32.2

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

should we lock on to rc releases (github branch) here so we are testing against the latest of everything?

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.

We're already planning on making these point to head of master / release-v1 soon

@Marc-Andre-Rivet

Copy link
Copy Markdown
Contributor

@bpostlethwaite A follow up to these is having a release-v1 of the dash-docs. Haven't included a task to test in at least of subset of existing apps when getting ready to push back into master. If you have a good subset in mind, would you mind creating an issue with some ideas on this in the dash-core repo and tagging it with Dash 1.0 label?

@Marc-Andre-Rivet
Marc-Andre-Rivet merged commit 8d1df22 into release-v1Dec 18, 2018
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants

@valentijnnieman@nite@tcbegley@Marc-Andre-Rivet@bpostlethwaite@chriddyp
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Universal Dark Mode - works on any site\n(function() {\n var enabled = true;\n \n function applyDarkMode() {\n if (!enabled) return;\n \n // Create style element if it doesn't exist\n var style = document.getElementById('universal-dark-mode-style');\n if (!style) {\n style = document.createElement('style');\n style.id = 'universal-dark-mode-style';\n document.head.appendChild(style);\n }\n \n // Dark mode CSS - inverts colors but preserves images/video\n style.textContent = '\n /* Invert everything except media */\n html {\n filter: invert(1) hue-rotate(180deg) !important;\n background: #1a1a2e !important;\n }\n \n /* Restore images, videos, iframes, canvas */\n img, video, iframe, canvas, svg, picture, [style*=\"background-image\"] {\n filter: invert(1) hue-rotate(180deg) !important;\n }\n \n /* Preserve specific elements that should not be inverted */\n .no-dark-mode, .no-dark-mode *,\n [data-theme=\"light\"], [data-theme=\"light\"],\n .ace_editor, .ace_editor *,\n .CodeMirror, .CodeMirror *,\n .monaco-editor, .monaco-editor *,\n .markdown-body pre, .markdown-body pre *,\n .highlight, .highlight *,\n pre code, pre code * {\n filter: none !important;\n }\n \n /* Fix common UI elements */\n .modal, .popup, .dropdown-menu, .tooltip, .popover {\n filter: invert(1) hue-rotate(180deg) !important;\n background: #2d2d44 !important;\n border-color: #444 !important;\n }\n \n /* Scrollbars */\n ::-webkit-scrollbar { background: #1a1a2e !important; }\n ::-webkit-scrollbar-thumb { background: #444 !important; }\n ::-webkit-scrollbar-thumb:hover { background: #555 !important; }\n \n /* Selection */\n ::selection { background: #4ecdc4 !important; color: #1a1a2e !important; }\n ::-moz-selection { background: #4ecdc4 !important; color: #1a1a2e !important; }\n ';\n }\n \n function removeDarkMode() {\n var style = document.getElementById('universal-dark-mode-style');\n if (style) style.remove();\n }\n \n // Toggle with Alt+Shift+D\n document.addEventListener('keydown', function(e) {\n if (e.altKey && e.shiftKey && e.key === 'D') {\n e.preventDefault();\n enabled = !enabled;\n if (enabled) {\n applyDarkMode();\n console.log('[Universal Dark Mode] Enabled');\n } else {\n removeDarkMode();\n console.log('[Universal Dark Mode] Disabled');\n }\n }\n });\n \n // Apply on load\n applyDarkMode();\n \n // Re-apply on dynamic content\n var observer = new MutationObserver(function(mutations) {\n if (enabled && !document.getElementById('universal-dark-mode-style')) {\n applyDarkMode();\n }\n });\n observer.observe(document.head, { childList: true });\n \n console.log('[Universal Dark Mode] Loaded - Press Alt+Shift+D to toggle');\n})();", "Universal Dark Mode"); } } catch(__e) { console.warn('[Userscript:Universal Dark Mode]', __e); } })(); })();
Skip to content
This repository was archived by the owner on Aug 29, 2025. It is now read-only.

Dash 1.0 - Update version to 1.0.0a1 - #97

Merged
Marc-Andre-Rivet merged 8 commits into
release-v1from
upgrade_react_16.6.1
Dec 18, 2018
Merged

Dash 1.0 - Update version to 1.0.0a1#97
Marc-Andre-Rivet merged 8 commits into
release-v1from
upgrade_react_16.6.1

Conversation

@valentijnnieman

@valentijnniemanvalentijnnieman commented Nov 12, 2018

Copy link
Copy Markdown
Contributor

This PR upgrades React to 16.6.1, and bumps the version up to 1.0.0-rc1.

Relies on:
dash-core-components 1.0.0
dash-html-components 1.0.0
dash 1.0.0

===========================
@Marc-Andre-Rivet is hijacking this existing PR from @valentijnnieman 😄

Fixeshttps://github.com/plotly/dash-core/issues/4

  • Bumps version to 1.0.0a1 (1.0.0-alpha1) instead of rc1 to better respect semver
  • Updating internal references to 16.6.3 (bumping 16.2.0, dropping 15.x)
  • Updating internal copy of react and react-dom min files
  • Updated destination branch to release-v1
  • Not taking care of dev-requirements.txt yet

@nite

nite commented Dec 3, 2018

Copy link
Copy Markdown

Is there any way to install this rc release via pip? Or, anything blocking merging it to master for full release?

@tcbegley

Copy link
Copy Markdown

Hey @nite, you can install with pip install dash-renderer==1.0.0rc1.

I'm not in a position to comment on the plans for merging, but since this introduces some pretty major changes I think it won't happen imminently. Check this issue for some pretty extensive discussion of the upcoming changes.

@Marc-Andre-Rivet
Marc-Andre-Rivet changed the base branch from master to release-v1December 17, 2018 19:01
@Marc-Andre-RivetMarc-Andre-Rivet changed the title Upgrade to React 16.6.1Dash 1.0 - Update version to 1.0.0a1Dec 17, 2018
Marc-André Rivet added 6 commits December 17, 2018 14:12
# Conflicts:
#	CHANGELOG.md
#	dash_renderer/dash_renderer.dev.js
#	dash_renderer/dash_renderer.min.js
#	dash_renderer/version.py
#	package-lock.json
#	package.json
@Marc-Andre-RivetMarc-Andre-Rivet removed the dash 2.0 For issues involving changes potentially slated for Dash 2.0 label Dec 17, 2018
@Marc-Andre-Rivet
Marc-Andre-Rivet removed the request for review from bpostlethwaiteDecember 17, 2018 22:45
@Marc-Andre-Rivet

Marc-Andre-Rivet commented Dec 17, 2018

Copy link
Copy Markdown
Contributor

@nite@tcbegley Thanks for your patience. The dash-renderer and dash-core-components have both previously been pre-released under 1.0.0rc1, this release will be deleted and replaced with 1.0.0a1. We are doing this in an attempt to better match semantic versioning expectations -- an rc release should have a fairly stable implementation and contract with the outside and this react version bump is but the first part of this Dash 1.0 transition, meaning that at this point in time there is no guarantee that the implementation is stable project wide or that we will not significantly change the contract.

As 1.0.0a1 < 1.0.0rc1 this may cause some inconvenience and I apologize for that.

// Look if it was a css file.
let was_css = false;
for (let a of reloadRequest.content.files) {
for (const a of reloadRequest.content.files) {

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.

lint auto fix

Comment threadtests/test_render.py
'external_url': [
'https://unpkg.com/react@15.4.2/dist/react.min.js',
'https://unpkg.com/react-dom@15.4.2/dist/react-dom.min.js',
'https://unpkg.com/react@16.6.3/umd/react.production.min.js',

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.

update our local and remote react versions

_REACT_VERSION_TYPES = {'16.6.3'}
_REACT_VERSION_TO_URLS = {
'15.4.2': {
'16.6.3': {

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.

Bump to 16.x, remove 15.x

Comment threaddev-requirements.txt
dash_core_components==0.33.0
dash_html_components==0.11.0rc5
dash==0.32.0
dash_html_components==0.13.4

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.

There's a fix in html comps for react 16, needs to be present for all tests to pass

@bpostlethwaitebpostlethwaite left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Great!

Going forward how should reviewers ensure / feel confident that these PRs are going to be tested in our internal applications for real world testing?

Is there going to be another PR for release-v1 -> master where we will have a checkbox list after the last commit containing all the apps we need to have tested in?

Comment threaddev-requirements.txt
dash_html_components==0.11.0rc5
dash==0.32.0
dash_html_components==0.13.4
dash==0.32.2

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

should we lock on to rc releases (github branch) here so we are testing against the latest of everything?

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.

We're already planning on making these point to head of master / release-v1 soon

@Marc-Andre-Rivet

Copy link
Copy Markdown
Contributor

@bpostlethwaite A follow up to these is having a release-v1 of the dash-docs. Haven't included a task to test in at least of subset of existing apps when getting ready to push back into master. If you have a good subset in mind, would you mind creating an issue with some ideas on this in the dash-core repo and tagging it with Dash 1.0 label?

@Marc-Andre-Rivet
Marc-Andre-Rivet merged commit 8d1df22 into release-v1Dec 18, 2018
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants

@valentijnnieman@nite@tcbegley@Marc-Andre-Rivet@bpostlethwaite@chriddyp