Skip to content

Conversation

@rozele
Copy link
Contributor

@rozele rozele commented Feb 21, 2018

react-native recently moved to the MIT license. In order to remove the BSD+Patents licensing from react-native-windows, we have to re-derive the source code we copied from react-native from the latest source code. This change re-derives the source code in the Libraries folder.

* This source code is licensed under the BSD-style license found in the
* LICENSE file in the root directory of this source tree. An additional grant
* of patent rights can be found in the PATENTS file in the same directory.
*
Copy link
Contributor Author

@rozele rozele Feb 21, 2018

Choose a reason for hiding this comment

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

Detailed review needed. #Closed

Copy link
Contributor Author

Choose a reason for hiding this comment

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

RNTester working correctly.


In reply to: 169645133 [](ancestors = 169645133)

/**
* Copyright (c) 2015-present, Facebook, Inc.
* All rights reserved.
*
Copy link
Contributor Author

@rozele rozele Feb 21, 2018

Choose a reason for hiding this comment

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

Detailed review needed. #Closed

Copy link
Contributor Author

Choose a reason for hiding this comment

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

RNTester still working properly.


In reply to: 169646847 [](ancestors = 169646847)

/**
* Copyright (c) 2015-present, Facebook, Inc.
* All rights reserved.
*
Copy link
Contributor Author

@rozele rozele Feb 21, 2018

Choose a reason for hiding this comment

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

Detailed review needed. #Closed

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Had to revert to enable auto-grow. Work remains to ensure TextInput can grow correctly.


In reply to: 169647991 [](ancestors = 169647991)

* of patent rights can be found in the PATENTS file in the same directory.
*
* @providesModule ViewPropTypes
* @flow
Copy link
Contributor Author

@rozele rozele Feb 21, 2018

Choose a reason for hiding this comment

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

Confirm e.g. manipulationModes still work. #Closed

Copy link
Contributor Author

Choose a reason for hiding this comment

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

SwipeableListView in RNTester still working.


In reply to: 169648288 [](ancestors = 169648288)

/**
* Copyright (c) 2015-present, Facebook, Inc.
* All rights reserved.
*
Copy link
Contributor Author

@rozele rozele Feb 21, 2018

Choose a reason for hiding this comment

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

Detailed review needed. #Closed

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The only difference between this and upstream is indexedDBEnabled. I'm not sure we need this feature.


In reply to: 169648900 [](ancestors = 169648900)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

All features in RNTester are working.


In reply to: 169956593 [](ancestors = 169956593,169648900)

* This source code is licensed under the BSD-style license found in the
* LICENSE file in the root directory of this source tree. An additional grant
* of patent rights can be found in the PATENTS file in the same directory.
*
Copy link
Contributor Author

@rozele rozele Feb 21, 2018

Choose a reason for hiding this comment

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

Detailed review needed. #Closed

Copy link
Contributor Author

Choose a reason for hiding this comment

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

RNTester still working.


In reply to: 169649082 [](ancestors = 169649082)

* This source code is licensed under the BSD-style license found in the
* LICENSE file in the root directory of this source tree. An additional grant
* of patent rights can be found in the PATENTS file in the same directory.
*
Copy link
Contributor Author

@rozele rozele Feb 21, 2018

Choose a reason for hiding this comment

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

Detailed review needed. #Closed

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Old version still working, more work to do to subscribe to the connectionChange event.


In reply to: 169649181 [](ancestors = 169649181)

callback: (requestId: number) => any
callback: (requestId: number) => any,
withCredentials: boolean
) {
Copy link
Contributor Author

@rozele rozele Feb 21, 2018

Choose a reason for hiding this comment

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

Remove for now, but we should add this. #Closed

Copy link
Contributor Author

Choose a reason for hiding this comment

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

RNTester working once removed.


In reply to: 169649317 [](ancestors = 169649317)

@@ -1,576 +0,0 @@
/**
* Copyright (c) 2015-present, Facebook, Inc.
* All rights reserved.
Copy link
Contributor Author

@rozele rozele Feb 21, 2018

Choose a reason for hiding this comment

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

Ensure Text still works. #Closed

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Text still works as expected.


In reply to: 169649482 [](ancestors = 169649482)

/**
* Copyright (c) 2015-present, Facebook, Inc.
* All rights reserved.
*
Copy link
Contributor Author

@rozele rozele Feb 21, 2018

Choose a reason for hiding this comment

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

Detailed review needed. #Closed

Copy link
Contributor Author

Choose a reason for hiding this comment

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

RNTester WebSocket example still working (tested both binary and text send/receive)


In reply to: 169649951 [](ancestors = 169649951)

/**
* Copyright (c) 2015-present, Facebook, Inc.
* All rights reserved.
*
Copy link
Contributor Author

@rozele rozele Feb 21, 2018

Choose a reason for hiding this comment

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

Check if #253 is now fixed. #Closed

Copy link
Contributor Author

Choose a reason for hiding this comment

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

No - still work to do here to support all kinds of headers.


In reply to: 169755860 [](ancestors = 169755860)

@rozele
Copy link
Contributor Author

rozele commented Feb 22, 2018

Add UNSAFE to all deprecated React lifecycle events.

@rozele
Copy link
Contributor Author

rozele commented Feb 22, 2018

Track issues for all new features to catch up with. #Closed

* This source code is licensed under the BSD-style license found in the
* LICENSE file in the root directory of this source tree. An additional grant
* of patent rights can be found in the PATENTS file in the same directory.
*
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Verify if this fixes #1613 (and removes the need for #1639)

@rozele rozele force-pushed the mit branch 3 times, most recently from cec812e to cd7064c Compare March 1, 2018 11:12
rozele added 3 commits March 5, 2018 10:26
react-native recently moved to the MIT license. In order to remove the BSD+Patents licensing from react-native-windows, we have to re-derive the source code we copied from react-native from the latest source code. This change re-derives the source code in the Libraries folder.
Many of these files never existed in react-native. In most cases, the copyright header was just copied from other files. This was incorrect, the header should have originally reflected a Microsoft copyright. This commit fixes that, and also rebases copy-paste code against Facebook's latest MIT-licensed code.
…w MIT

Updates the README and the LICENSE at the root folder to reflect that portions of the React Native Windows plugin derived from Facebook are copyright Facebook. Both derived and original contributions in React Native Windows are provided under the MIT license.
@rozele rozele merged commit b8135c5 into microsoft:master Mar 6, 2018
@rozele rozele deleted the mit branch March 6, 2018 13:19
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.

1 participant