Remove React Native vendor polyfills from BSON

XMLWordPrintableJSON

    • Type: Task
    • Resolution: Fixed
    • Priority: Unknown
    • bson-7.0.0
    • Affects Version/s: None
    • Component/s: BSON
    • 3
    • Not Needed
    • 🔵 Done
    • Not Needed
    • Hide

      1. What would you like to communicate to the user about this feature?
      2. Would you like the user to see examples of the syntax and/or executable code and its output?
      3. Which versions of the driver/connector does this apply to?

      Show
      1. What would you like to communicate to the user about this feature? 2. Would you like the user to see examples of the syntax and/or executable code and its output? 3. Which versions of the driver/connector does this apply to?
    • None
    • None
    • None
    • None
    • None
    • None

      Use Case

      As a Node driver engineer,
      I want to bump the minimum supported hermes engine and react native version that BSON supports to a version that includes TextEncoder, atob and btoa,
      So that I can remove the vendored polyfills of these three globals from BSON, and remove the separate BSON React Native bundle from BSON.

      User Experience

      • Users on older React Native / Hermes versions either:
        • Won't be able to upgrade to bson v7.
        • Will need to install polyfills for these globals.

      Dependencies

      • n/a

      Risks/Unknowns

      • n/a

      Acceptance Criteria

      Implementation Requirements

      • Remove the `vendor` folder and any associated tooling, bundling, etc from BSON.
      • Remove the custom Rollup configuration + additional tooling from BSON.
      • Update bson's package.json's export for react-native to use the commonjs bundle of BSON.

      Testing Requirements

      • Manually test that this version of BSON is still usable on React native.

      Documentation Requirements

      • Update the readme in js-bson to accurately reflect polyfill requirements for react native users.

      Follow Up Requirements

      • n/a

            Assignee:
            Bailey Pearson
            Reporter:
            Bailey Pearson
            Durran Jordan
            Votes:
            1 Vote for this issue
            Watchers:
            3 Start watching this issue

              Created:
              Updated:
              Resolved: