Closed peitschie closed 2 years ago
The code seems OK but I am unsure about the header license.
We always use the Apache-2.0 license header.
I know you are just copying the existing file contents from cordova-plugin-splashscreen
, but I am even surprised that someone approved that.
Also, the URL in the header comment, https://github.com/apache/cordova-platform
, does not exist.
I am not sure what should be done regarding the license but other then that, everything seems OK.
@erisu I'll just replace this with the existing license from the javascript source files.
Platforms affected
Affects TypeScript enabled projects only
Motivation and Context
Type definitions to allow programmatic control of the splashscreen are needed in
cordova-ios
now thatcordova-plugin-splashscreen
was obsoleted as part of cordova-ios 6.X platform update.Description
Import the current type definitions from
cordova-plugin-splashscreen
and expose them for use within a typescript project.Testing
I've imported this into an existing TypeScript-based application that has
cordova-plugin-splashscreen
removed, and ensured that TypeScript compilation produces no warnings or errorsChecklist
(platform)
if this change only applies to one platform (e.g.(android)
)