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

[eas-cli] [ENG-10334] Display device creation date #2092

Merged

Conversation

radoslawkrzemien
Copy link
Contributor

@radoslawkrzemien radoslawkrzemien commented Oct 18, 2023

Why

https://linear.app/expo/issue/ENG-10334/show-creation-time-of-device-record-in-ad-hoc-device-selection-prompt
Companion of: https://github.com/expo/universe/pull/13641

How

Updated function formatting the display value of AppleDevice to include the creation date

Test Plan

Added automated tests

Deployment

  • www
  • generate cli gql schema from prod
  • cli

@linear
Copy link

linear bot commented Oct 18, 2023

ENG-10334 Show creation time of device record in ad hoc device selection prompt

image.png

gabriel just added a device, and I had multiple unselected devices listed and I had no idea which one was his. It would have been easy to determine if I saw when the device was registered.

@radoslawkrzemien
Copy link
Contributor Author

/changelog-entry chore Display Apple device creation date when listing devices

@github-actions
Copy link

github-actions bot commented Oct 18, 2023

Size Change: +387 B (0%)

Total Size: 42.6 MB

Filename Size Change
./packages/eas-cli/dist/eas-linux-x64.tar.gz 42.6 MB +387 B (0%)

compressed-size-action

@codecov
Copy link

codecov bot commented Oct 18, 2023

Codecov Report

Merging #2092 (101382e) into main (474c95a) will increase coverage by 0.01%.
Report is 1 commits behind head on main.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main    #2092      +/-   ##
==========================================
+ Coverage   54.07%   54.07%   +0.01%     
==========================================
  Files         508      508              
  Lines       18669    18674       +5     
  Branches     3735     3934     +199     
==========================================
+ Hits        10094    10097       +3     
+ Misses       8554     7886     -668     
- Partials       21      691     +670     
Files Coverage Δ
...s-cli/src/credentials/android/api/GraphqlClient.ts 36.18% <ø> (ø)
...eas-cli/src/credentials/ios/actions/DeviceUtils.ts 65.22% <100.00%> (+6.13%) ⬆️
packages/eas-cli/src/graphql/generated.ts 100.00% <ø> (ø)
...s-cli/src/graphql/types/credentials/AppleDevice.ts 100.00% <ø> (ø)

... and 190 files with indirect coverage changes

@github-actions
Copy link

✅ Thank you for adding the changelog entry!

@radoslawkrzemien radoslawkrzemien merged commit aa14f21 into main Oct 18, 2023
9 checks passed
@radoslawkrzemien radoslawkrzemien deleted the @radoslawkrzemien/ENG-10334-display-device-creation-date branch October 18, 2023 15:52
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.

2 participants