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

[TIMOB-23452] iOS: Remove all deprecated API's with remove-version 6.0.0 #8028

Merged
merged 11 commits into from Jul 5, 2016

Conversation

hansemannn
Copy link
Collaborator

@hansemannn hansemannn commented May 28, 2016

JIRA: https://jira.appcelerator.org/browse/TIMOB-23452

Description:
This PR removes all API's that are marked to be removed in 6.0.0. It is structured to hold different proxy-changes in own commits to make it easier to review.

the playing video will fill the entire video player area, obscuring any background
view.
platforms: [iphone, ipad]
parameters:
Copy link
Contributor

Choose a reason for hiding this comment

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

This is generating an error when you do node validate.js in apidoc

[ERROR] ./Titanium/Media/VideoPlayer.yml
Titanium.Media.VideoPlayer: found 1 error(s)!
    properties
        backgroundView
            Invalid key: parameters

Most likely because it doesn't expect properties to contain parameters.

@cheekiatng
Copy link
Contributor

@hansemannn looks good. please fix conflicts and address comment.

@hansemannn
Copy link
Collaborator Author

Done, let's wait for the CI to finish.

@sgtcoolguy sgtcoolguy modified the milestone: 6.0.0 Jun 17, 2016
@sgtcoolguy sgtcoolguy removed the 6.0.0 label Jun 17, 2016
@cheekiatng
Copy link
Contributor

@hansemannn argh I'm sorry i forgot to merge. can you fix the conflicts.. again..

… TIMOB-23452

# Conflicts:
#	apidoc/Titanium/Media/Media.yml
@hansemannn
Copy link
Collaborator Author

Done

@cheekiatng
Copy link
Contributor

APPROVED

@cheekiatng cheekiatng merged commit 3c7a5cd into tidev:master Jul 5, 2016
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

3 participants