Skip to content

chore: packages update#23

Open
iordanGrancharov wants to merge 2 commits into
masterfrom
packages-update
Open

chore: packages update#23
iordanGrancharov wants to merge 2 commits into
masterfrom
packages-update

Conversation

@iordanGrancharov

Copy link
Copy Markdown

Short description:

  • updated all packages to the latest
  • fixed dependency conflicts
  • components are using svg icons only

Related items

@PekoPPT PekoPPT left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

We have to include a package-lock.json file with pre-defined versions of the components to be able to pass the build. Check the error here for more info: https://github.com/telerik/react-coffee-warehouse/actions/runs/7277313292/job/19829038754?pr=23

@PekoPPT PekoPPT left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

The app works OK and no warnings appear. The only thing I noticed is the following that appears in the the console, when the app is started locally:
image

When navigating to http://localhost:3000/react-coffee-warehouse the console warns the following:
image

As the app si automatically deployed this URL could be there on purpose but you can discuss it with @kspeyanski

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