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

fix(ios): can not show fullscreen modal windows anymore #11267

Merged
merged 9 commits into from Oct 14, 2019

Conversation

vijaysingh-axway
Copy link
Contributor

@build build added this to the 8.3.0 milestone Oct 8, 2019
@build build requested a review from a team October 8, 2019 22:51
@build
Copy link
Contributor

build commented Oct 8, 2019

Fails
🚫 Tests have failed, see below for more information.
Messages
📖

💾 Here's the generated SDK zipfile.

📖 ❌ 9 tests have failed There are 9 tests failing and 469 skipped out of 4591 total tests.
📖 ✊ The commits in this PR match our conventions! Feel free to Rebase and Merge this PR when ready.

Tests:

ClassnameNameTimeError
ios.Titanium.UI.LayoutTIMOB-23372 #100.002
Error: expected 9 to equal 10
fail@file:///Users/build/Library/Developer/CoreSimulator/Devices/642A82C6-A372-44D5-87DA-40B5A9A1F6E5/data/Containers/Bundle/Application/A830085C-6CB7-4B3D-A871-81AD93AC3FAC/mocha.app/node_modules/should/cjs/should.js:275:23
value@file:///Users/build/Library/Developer/CoreSimulator/Devices/642A82C6-A372-44D5-87DA-40B5A9A1F6E5/data/Containers/Bundle/Application/A830085C-6CB7-4B3D-A871-81AD93AC3FAC/mocha.app/node_modules/should/cjs/should.js:356:23
listener@file:///Users/build/Library/Developer/CoreSimulator/Devices/642A82C6-A372-44D5-87DA-40B5A9A1F6E5/data/Containers/Bundle/Application/A830085C-6CB7-4B3D-A871-81AD93AC3FAC/mocha.app/ti.ui.layout.test.js:1792:33
ios.Titanium.UI.LayoutTIMOB-23372 #90.001
Error: expected 9 to equal 10
fail@file:///Users/build/Library/Developer/CoreSimulator/Devices/642A82C6-A372-44D5-87DA-40B5A9A1F6E5/data/Containers/Bundle/Application/A830085C-6CB7-4B3D-A871-81AD93AC3FAC/mocha.app/node_modules/should/cjs/should.js:275:23
value@file:///Users/build/Library/Developer/CoreSimulator/Devices/642A82C6-A372-44D5-87DA-40B5A9A1F6E5/data/Containers/Bundle/Application/A830085C-6CB7-4B3D-A871-81AD93AC3FAC/mocha.app/node_modules/should/cjs/should.js:356:23
listener@file:///Users/build/Library/Developer/CoreSimulator/Devices/642A82C6-A372-44D5-87DA-40B5A9A1F6E5/data/Containers/Bundle/Application/A830085C-6CB7-4B3D-A871-81AD93AC3FAC/mocha.app/ti.ui.layout.test.js:1762:33
ios.Titanium.UI.LayoutTIMOB-23372 #80.002
Error: expected 9 to equal 10
fail@file:///Users/build/Library/Developer/CoreSimulator/Devices/642A82C6-A372-44D5-87DA-40B5A9A1F6E5/data/Containers/Bundle/Application/A830085C-6CB7-4B3D-A871-81AD93AC3FAC/mocha.app/node_modules/should/cjs/should.js:275:23
value@file:///Users/build/Library/Developer/CoreSimulator/Devices/642A82C6-A372-44D5-87DA-40B5A9A1F6E5/data/Containers/Bundle/Application/A830085C-6CB7-4B3D-A871-81AD93AC3FAC/mocha.app/node_modules/should/cjs/should.js:356:23
listener@file:///Users/build/Library/Developer/CoreSimulator/Devices/642A82C6-A372-44D5-87DA-40B5A9A1F6E5/data/Containers/Bundle/Application/A830085C-6CB7-4B3D-A871-81AD93AC3FAC/mocha.app/ti.ui.layout.test.js:1732:33
ios.Titanium.UI.LayoutTIMOB-23372 #70.002
Error: expected 9 to equal 10
fail@file:///Users/build/Library/Developer/CoreSimulator/Devices/642A82C6-A372-44D5-87DA-40B5A9A1F6E5/data/Containers/Bundle/Application/A830085C-6CB7-4B3D-A871-81AD93AC3FAC/mocha.app/node_modules/should/cjs/should.js:275:23
value@file:///Users/build/Library/Developer/CoreSimulator/Devices/642A82C6-A372-44D5-87DA-40B5A9A1F6E5/data/Containers/Bundle/Application/A830085C-6CB7-4B3D-A871-81AD93AC3FAC/mocha.app/node_modules/should/cjs/should.js:356:23
listener@file:///Users/build/Library/Developer/CoreSimulator/Devices/642A82C6-A372-44D5-87DA-40B5A9A1F6E5/data/Containers/Bundle/Application/A830085C-6CB7-4B3D-A871-81AD93AC3FAC/mocha.app/ti.ui.layout.test.js:1699:33
ios.Titanium.UI.LayoutTIMOB-23372 #60.002
Error: expected 9 to equal 10
fail@file:///Users/build/Library/Developer/CoreSimulator/Devices/642A82C6-A372-44D5-87DA-40B5A9A1F6E5/data/Containers/Bundle/Application/A830085C-6CB7-4B3D-A871-81AD93AC3FAC/mocha.app/node_modules/should/cjs/should.js:275:23
value@file:///Users/build/Library/Developer/CoreSimulator/Devices/642A82C6-A372-44D5-87DA-40B5A9A1F6E5/data/Containers/Bundle/Application/A830085C-6CB7-4B3D-A871-81AD93AC3FAC/mocha.app/node_modules/should/cjs/should.js:356:23
listener@file:///Users/build/Library/Developer/CoreSimulator/Devices/642A82C6-A372-44D5-87DA-40B5A9A1F6E5/data/Containers/Bundle/Application/A830085C-6CB7-4B3D-A871-81AD93AC3FAC/mocha.app/ti.ui.layout.test.js:1654:33
ios.Titanium.UI.LayoutTIMOB-23372 #50.002
Error: expected 9 to equal 10
fail@file:///Users/build/Library/Developer/CoreSimulator/Devices/642A82C6-A372-44D5-87DA-40B5A9A1F6E5/data/Containers/Bundle/Application/A830085C-6CB7-4B3D-A871-81AD93AC3FAC/mocha.app/node_modules/should/cjs/should.js:275:23
value@file:///Users/build/Library/Developer/CoreSimulator/Devices/642A82C6-A372-44D5-87DA-40B5A9A1F6E5/data/Containers/Bundle/Application/A830085C-6CB7-4B3D-A871-81AD93AC3FAC/mocha.app/node_modules/should/cjs/should.js:356:23
listener@file:///Users/build/Library/Developer/CoreSimulator/Devices/642A82C6-A372-44D5-87DA-40B5A9A1F6E5/data/Containers/Bundle/Application/A830085C-6CB7-4B3D-A871-81AD93AC3FAC/mocha.app/ti.ui.layout.test.js:1609:33
ios.Titanium.UI.LayoutTIMOB-23372 #40.002
Error: expected 9 to equal 10
fail@file:///Users/build/Library/Developer/CoreSimulator/Devices/642A82C6-A372-44D5-87DA-40B5A9A1F6E5/data/Containers/Bundle/Application/A830085C-6CB7-4B3D-A871-81AD93AC3FAC/mocha.app/node_modules/should/cjs/should.js:275:23
value@file:///Users/build/Library/Developer/CoreSimulator/Devices/642A82C6-A372-44D5-87DA-40B5A9A1F6E5/data/Containers/Bundle/Application/A830085C-6CB7-4B3D-A871-81AD93AC3FAC/mocha.app/node_modules/should/cjs/should.js:356:23
listener@file:///Users/build/Library/Developer/CoreSimulator/Devices/642A82C6-A372-44D5-87DA-40B5A9A1F6E5/data/Containers/Bundle/Application/A830085C-6CB7-4B3D-A871-81AD93AC3FAC/mocha.app/ti.ui.layout.test.js:1566:33
ios.Titanium.UI.LayoutTIMOB-23372 #30.002
Error: expected 9 to equal 10
fail@file:///Users/build/Library/Developer/CoreSimulator/Devices/642A82C6-A372-44D5-87DA-40B5A9A1F6E5/data/Containers/Bundle/Application/A830085C-6CB7-4B3D-A871-81AD93AC3FAC/mocha.app/node_modules/should/cjs/should.js:275:23
value@file:///Users/build/Library/Developer/CoreSimulator/Devices/642A82C6-A372-44D5-87DA-40B5A9A1F6E5/data/Containers/Bundle/Application/A830085C-6CB7-4B3D-A871-81AD93AC3FAC/mocha.app/node_modules/should/cjs/should.js:356:23
listener@file:///Users/build/Library/Developer/CoreSimulator/Devices/642A82C6-A372-44D5-87DA-40B5A9A1F6E5/data/Containers/Bundle/Application/A830085C-6CB7-4B3D-A871-81AD93AC3FAC/mocha.app/ti.ui.layout.test.js:1522:33
ios.Titanium.UI.LayoutTIMOB-23372 #20.002
Error: expected 9 to equal 10
fail@file:///Users/build/Library/Developer/CoreSimulator/Devices/642A82C6-A372-44D5-87DA-40B5A9A1F6E5/data/Containers/Bundle/Application/A830085C-6CB7-4B3D-A871-81AD93AC3FAC/mocha.app/node_modules/should/cjs/should.js:275:23
value@file:///Users/build/Library/Developer/CoreSimulator/Devices/642A82C6-A372-44D5-87DA-40B5A9A1F6E5/data/Containers/Bundle/Application/A830085C-6CB7-4B3D-A871-81AD93AC3FAC/mocha.app/node_modules/should/cjs/should.js:356:23
listener@file:///Users/build/Library/Developer/CoreSimulator/Devices/642A82C6-A372-44D5-87DA-40B5A9A1F6E5/data/Containers/Bundle/Application/A830085C-6CB7-4B3D-A871-81AD93AC3FAC/mocha.app/ti.ui.layout.test.js:1477:33

Generated by 🚫 dangerJS against d479081

@ssekhri
Copy link

ssekhri commented Oct 10, 2019

FR Passed. Modal windows shown properly as per the style.
Verified on:
Mac OS: 10.14.5
SDK: 8.3.0.v20191010111000
Appc CLI: 7.1.1
JDK: 1.8.0_162
Node: 10.5.0
Studio: 5.1.4.201909061933
Xcode: 11.0
Device: iPhone X(13.0), iPhone 7Plus(12.3.1)

@sgtcoolguy sgtcoolguy merged commit 1e3d161 into tidev:master Oct 14, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants