Skip to content

Re-merge "Always expect a channel in project.js"#4911

Merged
joshlory merged 4 commits into
stagingfrom
remerge-project-expect-channel
Oct 29, 2015
Merged

Re-merge "Always expect a channel in project.js"#4911
joshlory merged 4 commits into
stagingfrom
remerge-project-expect-channel

Conversation

@joshlory

Copy link
Copy Markdown
Contributor

Reverts #4905. I've fixed the regression, added a UI test and re-merged the changes from #4822.

@Bjvanminnen

Copy link
Copy Markdown
Contributor

lgtm

@joshlory

Copy link
Copy Markdown
Contributor Author

UI test passing via Sauce Connect:

./runner.rb -d localhost-studio.code.org:3000 --html -f features/legacyShareRemix.feature --parallel 12

UI tests for Firefox39Win7_legacyShareRemix succeeded (2:31 minutes, 1 passed)
UI tests for Firefox38Yosemite_legacyShareRemix succeeded (2:41 minutes, 1 passed)
UI tests for IE10Win7_legacyShareRemix succeeded (2:50 minutes, 1 passed)
UI tests for Chrome44XP_legacyShareRemix succeeded (3:34 minutes, 1 passed)
UI tests for ChromeLatestWin7_legacyShareRemix succeeded (3:36 minutes, 1 passed)
UI tests for SafariYosemite_legacyShareRemix succeeded (3:38 minutes, 1 passed)
UI tests for IE9Win7_legacyShareRemix succeeded (1:19 minutes, 1 passed)
UI tests for iPad_legacyShareRemix succeeded (4:30 minutes, 1 passed)

joshlory pushed a commit that referenced this pull request Oct 29, 2015
…nnel

Re-merge "Always expect a channel in `project.js`"
@joshlory joshlory merged commit 7460930 into staging Oct 29, 2015
@joshlory joshlory deleted the remerge-project-expect-channel branch October 29, 2015 00:19
deploy-code-org added a commit that referenced this pull request Oct 29, 2015
16bc449 Merge pull request #4903 from code-dot-org/only_setup_local_pegasus_db (philbogle)
7460930 Merge pull request #4911 from code-dot-org/remerge-project-expect-channel (Josh Lory)
4907658 Merge pull request #4915 from code-dot-org/revert-4723-ajax_sign_in (philbogle)
c266d62 Revert "Render the sign-in/sign-out button in the Dashboard header using Ajax" (philbogle)
0dc2fb9 Header (and 'Remix' button) doesn't show up on mobile [ci skip] (Josh Lory)
7317599 Merge pull request #4913 from code-dot-org/all-responses (Laurel)
9ac35c2 Merge pull request #4723 from code-dot-org/ajax_sign_in (philbogle)
deploy-code-org added a commit that referenced this pull request Oct 29, 2015
224dc67 Merge pull request #4786 from code-dot-org/pulse-item (Brad Buchanan)
d619792 Automatically built. (Continuous Integration)
8403a32 Merge pull request #4920 from code-dot-org/testfixes (Elijah Hamovitz)
0ff348c Temporarily removing funOMeter test (Elijah Hamovitz)
b457e24 Merge pull request #4916 from code-dot-org/ie10tests (Bjvanminnen)
5b89e35 more IE10 (Brent Van Minnen)
1876a7f o Merge branch 'staging' of github.com:code-dot-org/code-dot-org into staging (Trevor Berg)
541fdc5 Fix issues with JSONFileDataAdapter (Trevor Berg)
0f13914 iOS requires directly-bound click events and 'cursor: pointer' style for click events to work (Elijah Hamovitz)
5b6b40f Feature-check SVG filter support before creating/applying a filter. (Brad Buchanan)
656f7c2 Merge pull request #4877 from code-dot-org/bundle_install_frontends (Will Jordan)
b7be14d Automatically built. (Continuous Integration)
16bc449 Merge pull request #4903 from code-dot-org/only_setup_local_pegasus_db (philbogle)
7460930 Merge pull request #4911 from code-dot-org/remerge-project-expect-channel (Josh Lory)
f53452b start running applab tests on IE 10 (Brent Van Minnen)
4907658 Merge pull request #4915 from code-dot-org/revert-4723-ajax_sign_in (philbogle)
c266d62 Revert "Render the sign-in/sign-out button in the Dashboard header using Ajax" (philbogle)
0dc2fb9 Header (and 'Remix' button) doesn't show up on mobile [ci skip] (Josh Lory)
7317599 Merge pull request #4913 from code-dot-org/all-responses (Laurel)
3130e6c Merge remote-tracking branch 'origin/staging' into pulse-item (Brad Buchanan)
e28f355 Deduplicate functions and remove last SMIL TODO comment. (Brad Buchanan)
9ac35c2 Merge pull request #4723 from code-dot-org/ajax_sign_in (philbogle)
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