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

chore(ios) : Updated ios module’s version to latest #10870

Merged
merged 11 commits into from Jun 19, 2019

Conversation

vijaysingh-axway
Copy link
Contributor

@build
Copy link
Contributor

build commented May 1, 2019

Messages
📖

💾 Here's the generated SDK zipfile.

📖

✅ All tests are passing
Nice one! All 3615 tests are passing.
(There are 464 tests skipped)

Generated by 🚫 dangerJS against 41cbe04

},
"ti.safaridialog": {
"url": "https://github.com/appcelerator-modules/ti.safaridialog/releases/download/iOS-1.1.1/ti.safaridialog-iphone-1.1.1.zip",
"integrity": "sha512-6g3Qi9RUCl3uf06OnrThm9fpxhcxhtRzyoFRE5PJpenTFN45QYGRZeEQAqgTKcvEB0ardUfGMOyZcA8WoCfWTw=="
"url": "https://github.com/appcelerator-modules/ti.safaridialog/releases/download/iOS-2.0.0/ti.safaridialog-iphone-2.0.0.zip",
Copy link
Collaborator

Choose a reason for hiding this comment

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

Can we include a new major of a module in 8.1.0?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Modules have their own release cycle, other than sdk. So I think it should not be a problem including major of module in 8.1.0. @mukherjee2 @lokeshchdhry ?

Copy link
Contributor

Choose a reason for hiding this comment

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

@vijaysingh-axway By default we always use the latest version of a module of no explicit version is specified in tiapp.xml. Now, if the ti.safaridialog module contains a breaking API change this will break existing projects.

@ewanharris already opened a ticket for this issue, i just can't find it right now.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@ewanharris @janvennemann Restored ti.safaridialog to non-major version.

@sgtcoolguy sgtcoolguy modified the milestones: 8.1.0, 8.2.0 Jun 3, 2019
@keerthi1032
Copy link
Contributor

@vijaysingh-axway ti.safaridialog module still has the version 1.1.1 with the PR's SDK. Not getting the Latest version. Can you please look at it

@vijaysingh-axway
Copy link
Contributor Author

@vijaysingh-axway ti.safaridialog module still has the version 1.1.1 with the PR's SDK. Not getting the Latest version. Can you please look at it

@keerthi1032 ti.safaridialog has major release version 2.0.0. So that will be included in SDK's Major release.

@keerthi1032
Copy link
Contributor

@vijaysingh-axway ti.safaridialog module still has the version 1.1.1 with the PR's SDK. Not getting the Latest version. Can you please look at it

@keerthi1032 ti.safaridialog has major release version 2.0.0. So that will be included in SDK's Major release.
Ok Thanks for the information

@keerthi1032
Copy link
Contributor

@vijaysingh-axway ti.safaridialog module still has the version 1.1.1 with the PR's SDK. Not getting the Latest version. Can you please look at it

@keerthi1032 ti.safaridialog has major release version 2.0.0. So that will be included in SDK's Major release.
@vijaysingh-axway Ok Thanks for the information

@keerthi1032
Copy link
Contributor

FR passed. Latest version updated for default modules on iOS side.

Test Environment:
Name = Mac OS X
Version = 10.13.6
Architecture = 64bit
Node.js
Node.js Version = 10.13.0
npm Version = 6.4.1
Titanium CLI
CLI Version = 5.1.1
Titanium SDK
SDK Version = local sdk 8.2.0.v20190612152208
CLI =7.0.12-70X.5

@keerthi1032 keerthi1032 merged commit b9f1206 into tidev:master Jun 19, 2019
hansemannn pushed a commit to hansemannn/titanium_mobile that referenced this pull request Jul 8, 2019
* chore(android): bump ti.playservices to 16.1.3

* fix: fix download method and cleanup modules-integrity

* chore(android): update ti.playservices hash

* chore(ios) : Updated ios module’s version to latest

* fix(ios) : ti.safaridialog restored to non-major version as per review comment.
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

7 participants