-
Notifications
You must be signed in to change notification settings - Fork 0
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
[CU] July Dependency Updates #405
Changes from 13 commits
9c55f6c
483a733
866c1c8
2ad1349
0ffe492
eabd41b
c3d172a
446fff7
7356990
77d675c
f5cf622
81620ff
7880410
b9888a6
d96b8e1
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -18,15 +18,15 @@ | |
"homepage": "https://github.com/department-of-veterans-affairs/va-mobile-library#readme", | ||
"packageManager": "[email protected]", | ||
"devDependencies": { | ||
"@typescript-eslint/eslint-plugin": "^7.0.1", | ||
"@typescript-eslint/parser": "^7.0.1", | ||
"eslint": "^8.56.0", | ||
"@typescript-eslint/eslint-plugin": "^7.17.0", | ||
"@typescript-eslint/parser": "^7.17.0", | ||
"eslint": "^8.57.0", | ||
"eslint-config-prettier": "^9.1.0", | ||
"eslint-plugin-sort-imports-es6-autofix": "^0.6.0", | ||
"eslint-plugin-testing-library": "^6.2.2", | ||
"eslint-plugin-tsdoc": "^0.2.17", | ||
"prettier": "3.2.5", | ||
"typescript": "^5.3.3" | ||
"eslint-plugin-tsdoc": "^0.3.0", | ||
"prettier": "3.3.3", | ||
"typescript": "~5.3.3" | ||
}, | ||
"resolutions": { | ||
"@storybook/react-docgen-typescript-plugin": "1.0.6--canary.9.cd77847.0", | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -18,6 +18,7 @@ module.exports = { | |
}, | ||
core: { | ||
builder: 'webpack5', | ||
disableWhatsNewNotifications: true, | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Not related to dependency updates but kept seeing this pop up as I was testing and decided to disable it. |
||
}, | ||
framework: '@storybook/react-webpack5', | ||
options: { builder: { useSWC: true } }, // improves build performance | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,6 +1,6 @@ | ||
{ | ||
"name": "@department-of-veterans-affairs/mobile-component-library", | ||
"version": "0.20.4-alpha.0", | ||
"version": "0.20.4-alpha.2", | ||
"description": "VA Design System Mobile Component Library", | ||
"main": "src/index.tsx", | ||
"scripts": { | ||
|
@@ -39,12 +39,12 @@ | |
}, | ||
"homepage": "https://department-of-veterans-affairs.github.io/va-mobile-library", | ||
"dependencies": { | ||
"@os-team/i18next-react-native-language-detector": "^1.0.31", | ||
"i18next": "^23.11.5", | ||
"react-i18next": "^14.1.2", | ||
"@os-team/i18next-react-native-language-detector": "^1.0.34", | ||
"i18next": "^23.12.2", | ||
"react-i18next": "^15.0.0", | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Probably fine, but may be good to double check use cases in the app if you didn't since this is the package that broke from a minor update recently. So double checking the external link prompt to leave the app and "# of #" screen reading for Segmented control. Good confirmation they didn't do anything breaking with a major increment since they broke things with a minor increment. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Confirmed with voiceover in the app that these translations are still working properly. |
||
"react-native-svg": "15.2.0", | ||
"react-native-toast-notifications": "^3.4.0", | ||
"styled-components": "^6.1.8" | ||
"styled-components": "^6.1.12" | ||
}, | ||
"peerDependencies": { | ||
"@department-of-veterans-affairs/mobile-assets": "^0.12.0", | ||
|
@@ -54,65 +54,65 @@ | |
"react-native-gesture-handler": "^2.12.0" | ||
}, | ||
"devDependencies": { | ||
"@babel/core": "^7.24.0", | ||
"@babel/plugin-transform-react-jsx": "^7.23.4", | ||
"@babel/preset-env": "^7.23.9", | ||
"@babel/preset-typescript": "^7.23.3", | ||
"@babel/core": "^7.24.9", | ||
"@babel/plugin-transform-react-jsx": "^7.24.7", | ||
"@babel/preset-env": "^7.24.8", | ||
"@babel/preset-typescript": "^7.24.7", | ||
"@department-of-veterans-affairs/mobile-assets": "0.12.0", | ||
"@department-of-veterans-affairs/mobile-tokens": "0.13.0", | ||
"@expo/metro-runtime": "~3.2.1", | ||
"@expo/webpack-config": "~19.0.1", | ||
"@react-native-async-storage/async-storage": "1.23.1", | ||
"@react-native-community/datetimepicker": "7.7.0", | ||
"@react-native-community/datetimepicker": "8.0.1", | ||
"@react-native-community/slider": "4.5.2", | ||
"@storybook/addon-actions": "7.6.19", | ||
"@storybook/addon-controls": "7.6.19", | ||
"@storybook/addon-actions": "7.6.20", | ||
"@storybook/addon-controls": "7.6.20", | ||
"@storybook/addon-designs": "^7.0.9", | ||
"@storybook/addon-docs": "7.6.19", | ||
"@storybook/addon-essentials": "7.6.19", | ||
"@storybook/addon-links": "7.6.19", | ||
"@storybook/addon-ondevice-actions": "^7.6.18", | ||
"@storybook/addon-ondevice-controls": "^7.6.18", | ||
"@storybook/addon-react-native-web": "^0.0.23", | ||
"@storybook/builder-webpack5": "7.6.19", | ||
"@storybook/react": "7.6.19", | ||
"@storybook/react-native": "^7.6.18", | ||
"@storybook/react-webpack5": "7.6.19", | ||
"@storybook/addon-docs": "7.6.20", | ||
"@storybook/addon-essentials": "7.6.20", | ||
"@storybook/addon-links": "7.6.20", | ||
"@storybook/addon-ondevice-actions": "^7.6.20", | ||
"@storybook/addon-ondevice-controls": "^7.6.20", | ||
"@storybook/addon-react-native-web": "^0.0.24", | ||
"@storybook/builder-webpack5": "7.6.20", | ||
"@storybook/react": "7.6.20", | ||
"@storybook/react-native": "^7.6.20", | ||
"@storybook/react-webpack5": "7.6.20", | ||
"@svgr/webpack": "^8.1.0", | ||
"@testing-library/react-native": "^12.4.5", | ||
"@testing-library/react-native": "^12.5.1", | ||
"@types/jest": "^29.5.12", | ||
"@types/react": "~18.2.79", | ||
"@types/react-test-renderer": "^18.0.7", | ||
"@types/react-test-renderer": "^18.3.0", | ||
"@types/styled-components-react-native": "^5.2.5", | ||
"babel-jest": "^29.7.0", | ||
"babel-loader": "^9.1.3", | ||
"babel-plugin-react-docgen-typescript": "^1.5.1", | ||
"babel-plugin-react-native-web": "^0.19.10", | ||
"babel-plugin-react-native-web": "^0.19.12", | ||
"copy-webpack-plugin": "^12.0.2", | ||
"expo": "^51.0.4", | ||
"expo-constants": "~16.0.1", | ||
"expo-font": "~12.0.5", | ||
"expo": "^51.0.22", | ||
"expo-constants": "~16.0.2", | ||
"expo-font": "~12.0.9", | ||
"expo-linking": "~6.3.1", | ||
"expo-splash-screen": "~0.27.4", | ||
"expo-splash-screen": "~0.27.5", | ||
"expo-status-bar": "~1.12.1", | ||
"gh-pages": "^6.1.1", | ||
"jest": "^29.7.0", | ||
"jest-expo": "~51.0.1", | ||
"jest-expo": "~51.0.3", | ||
"jest-styled-components": "^7.2.0", | ||
"metro-react-native-babel-preset": "^0.77.0", | ||
"react": "^18.2.0", | ||
"react-dom": "18.2.0", | ||
"react-native": "0.74.1", | ||
"react-native": "0.74.3", | ||
"react-native-gesture-handler": "~2.16.1", | ||
"react-native-reanimated": "3.10.1", | ||
"react-native-safe-area-context": "4.10.1", | ||
"react-native-safe-area-context": "4.10.5", | ||
"react-native-screens": "3.31.1", | ||
"react-native-svg-transformer": "^1.3.0", | ||
"react-native-web": "~0.19.10", | ||
"react-test-renderer": "^18.2.0", | ||
"storybook": "7.6.19", | ||
"react-native-svg-transformer": "^1.5.0", | ||
"react-native-web": "~0.19.12", | ||
"react-test-renderer": "^18.3.1", | ||
"storybook": "7.6.20", | ||
"storybook-dark-mode": "^3.0.3", | ||
"ts-jest": "^29.1.2", | ||
"typescript": "^5.3.3" | ||
"ts-jest": "^29.2.3", | ||
"typescript": "~5.3.3" | ||
} | ||
} |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,3 +1,4 @@ | ||
// @ts-nocheck | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. FYI @TimRoe, Had to add this because when I tried testing in the app, I would get the following TS error:
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Hmmm, that's fine for this ticket but we should figure out something better long term where one doesn't need the config in global.d.ts to not get errors when not even using the component. Perhaps create a ticket just for that nearer the end of Snackbar implementation to figure out the least invasive way to go about it. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Yeah, I was implying that this is a temporary fix and assume we'd fix it as part of the implementation. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Created ticket to make sure we don't miss this: #412 |
||
import { ToastOptions } from 'react-native-toast-notifications/lib/typescript/toast' | ||
import React from 'react' | ||
import Toast, { ToastType } from 'react-native-toast-notifications' | ||
|
@@ -42,14 +43,16 @@ type SnackbarProviderProps = { | |
* <SnackbarProvider /> | ||
* </> | ||
* ``` | ||
* | ||
* | ||
* This config will allow it to be called anywhere including outside React components. | ||
* | ||
* | ||
* The Snackbar remains open indefinitely. App configuration should ensure it is dismissed on navigation. | ||
*/ | ||
export const SnackbarProvider: React.FC<SnackbarProviderProps> = ({ | ||
// globalSnackbar, | ||
}) => { | ||
export const SnackbarProvider: React.FC<SnackbarProviderProps> = ( | ||
{ | ||
// globalSnackbar, | ||
}, | ||
) => { | ||
Comment on lines
+51
to
+55
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Autoformatting |
||
return ( | ||
<Toast | ||
animationDuration={100} | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -30,8 +30,8 @@ | |
"homepage": "https://github.com/department-of-veterans-affairs/va-mobile-library#readme", | ||
"packageManager": "[email protected]", | ||
"devDependencies": { | ||
"@department-of-veterans-affairs/css-library": "0.8.4", | ||
"prettier": "^3.3.2", | ||
"@department-of-veterans-affairs/css-library": "0.8.5", | ||
"prettier": "^3.3.3", | ||
"style-dictionary": "^3.9.2" | ||
} | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why'd this go
^
to~
?There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is to satisfy the following expo warning:
Leaving it as a carat will install
v5.5.4
which causes the warning to reappear.