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

[CU] July Dependency Updates #405

Merged
merged 15 commits into from
Jul 25, 2024
Merged
Show file tree
Hide file tree
Changes from 13 commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
12 changes: 6 additions & 6 deletions package.json
Original file line number Diff line number Diff line change
Expand Up @@ -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"
Copy link
Contributor

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 ~?

Copy link
Contributor Author

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:

The following packages should be updated for best compatibility with the installed expo version:
  [email protected] - expected version: ~5.3.3

Leaving it as a carat will install v5.5.4 which causes the warning to reappear.

},
"resolutions": {
"@storybook/react-docgen-typescript-plugin": "1.0.6--canary.9.cd77847.0",
Expand Down
2 changes: 1 addition & 1 deletion packages/assets/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@
},
"homepage": "https://github.com/department-of-veterans-affairs/va-mobile-library#readme",
"devDependencies": {
"@department-of-veterans-affairs/component-library": "^42.2.1",
"@department-of-veterans-affairs/component-library": "^43.0.1",
"hast-util-to-html": "^9.0.1"
}
}
1 change: 1 addition & 0 deletions packages/components/.storybook/web/main.ts
Original file line number Diff line number Diff line change
Expand Up @@ -18,6 +18,7 @@ module.exports = {
},
core: {
builder: 'webpack5',
disableWhatsNewNotifications: true,
Copy link
Contributor Author

Choose a reason for hiding this comment

The 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
Expand Down
76 changes: 38 additions & 38 deletions packages/components/package.json
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": {
Expand Down Expand Up @@ -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",
Copy link
Contributor

Choose a reason for hiding this comment

The 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.

Copy link
Contributor Author

Choose a reason for hiding this comment

The 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",
Expand All @@ -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"
}
}
13 changes: 8 additions & 5 deletions packages/components/src/components/Snackbar/Snackbar.tsx
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
// @ts-nocheck
Copy link
Contributor Author

Choose a reason for hiding this comment

The 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:

node_modules/@department-of-veterans-affairs/mobile-component-library/src/components/Snackbar/Snackbar.tsx(57,34): error TS7017: Element implicitly has an 'any' type because type 'typeof globalThis' has no index signature. 
error Command failed with exit code 2.

Copy link
Contributor

Choose a reason for hiding this comment

The 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.

Copy link
Contributor Author

Choose a reason for hiding this comment

The 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.

Copy link
Contributor Author

Choose a reason for hiding this comment

The 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'
Expand Down Expand Up @@ -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
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Autoformatting

return (
<Toast
animationDuration={100}
Expand Down
4 changes: 2 additions & 2 deletions packages/linting/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -24,12 +24,12 @@
},
"homepage": "https://github.com/department-of-veterans-affairs/va-mobile-library#readme",
"dependencies": {
"eslint-plugin-deprecate": "^0.8.4"
"eslint-plugin-deprecate": "^0.8.5"
},
"peerDependencies": {
"eslint": ">= 8"
},
"devDependencies": {
"eslint": "^8.56.0"
"eslint": "^8.57.0"
}
}
4 changes: 2 additions & 2 deletions packages/tokens/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -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"
}
}
Loading
Loading