Skip to content

Conversation

Simek
Copy link
Contributor

@Simek Simek commented Feb 1, 2023

Why

Fixes:

How

Fill out missing comments for DocumentResult type based on the code and Git History.

Regenerate docs data for unversioned and latest SDK.

Test Plan

Package checks are passing, example in NCL works.

Checklist

@Simek Simek requested a review from EvanBacon February 1, 2023 13:50
@expo-bot expo-bot added the bot: suggestions ExpoBot has some suggestions label Feb 1, 2023
@expo-bot
Copy link
Collaborator

expo-bot commented Feb 1, 2023

Hi there! 👋 I'm a bot whose goal is to ensure your contributions meet our guidelines.

I've found some issues in your pull request that should be addressed (click on them for more details) 👇

⚠️ Suggestion: Missing changelog entries


Your changes should be noted in the changelog. Read Updating Changelogs guide and consider adding an appropriate entry to the following changelogs:


Generated by ExpoBot 🤖 against e1cfdf9

@Simek Simek merged commit e47dbfd into main Feb 2, 2023
@Simek Simek deleted the @simek/document-picker-fix-docs branch February 2, 2023 09:45
@amandeepmittal amandeepmittal changed the title [document-picker] fill missing descriptions in DocumentResult type [document-picker][docs] fill missing descriptions in DocumentResult type Feb 2, 2023
thecodedrift added a commit to thecodedrift/expo that referenced this pull request Feb 2, 2023
…flipper

* upstream/main: (47 commits)
  [docs] Update Hermes guide to state that Hermes is the new default engine (expo#21047)
  chore: don't mark issues with the "Issue accepted" label as stale (expo#21058)
  Switch default JS engine to Hermes (expo#21001)
  [mail-composer][android] fix composeAsync not resolving after send/ discard (expo#20869)
  Update CHANGELOG.md (expo#21061)
  [core][iOS] Fix expo modules aren't added to global (expo#21037)
  [test-suite] Fix import in the Image example (expo#21043)
  [test-suite] fix video hanging (expo#21057)
  [av][ncl][go] fix audio and video qa issues (expo#21055)
  [tools] Selecting pull requests to label in the publish command (expo#20991)
  [document-picker] fill missing descriptions in `DocumentResult` type (expo#21040)
  [tools] Bump http-cache-semantics from 4.1.0 to 4.1.1 (expo#21049)
  Bump http-cache-semantics from 4.1.0 to 4.1.1 in /docs (expo#21050)
  [apps][yarn-workspace] replace deprecated activateKeepAwake
  update changelogs for react-native 0.71 upgrade (expo#20858)
  Upgrade react native to 0.71.2 (expo#21045)
  [go] update @shopify/react-native-skia to 0.1.172 (expo#21014)
  [stripe] Upgrade stripe to 0.23.1 (expo#20964)
  [expo-firebase-*] Remove libraries (expo#20979)
  [docs] Update expo-secure-store to add info about Export compliance (expo#21021)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bot: suggestions ExpoBot has some suggestions
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[docs] missing clarity around file result from DocumentPicker.getDocumentAsync
3 participants