Skip to content

fix/242: update packages verions and replace old code#3

Merged
shribakb1 merged 1 commit into
mainfrom
fix/242/update_deprecated_packages
Dec 24, 2025
Merged

fix/242: update packages verions and replace old code#3
shribakb1 merged 1 commit into
mainfrom
fix/242/update_deprecated_packages

Conversation

@shribakb1
Copy link
Copy Markdown
Contributor

@shribakb1 shribakb1 commented Dec 24, 2025

dev

Code reviewers

Second Level Review

Summary of issue

The project was using deprecated Sass APIs and syntax, which caused multiple deprecation warnings during build time.

Summary of change

Summary of change

Updated shared Sass utilities to use modern, namespaced Sass modules

Replaced deprecated global functions (unit) with their recommended equivalents (math.unit)

Added explicit usage of Sass core modules via @use

Eliminated repeated deprecation warnings across all dependent components

Improved long-term compatibility with upcoming Dart Sass releases

No visual or functional styling changes were introduced.

Testing approach

Ran the application locally and verified successful build

Confirmed that Sass deprecation warnings related to global functions were resolved

Manually checked affected UI components to ensure styles remained unchanged

CHECK LIST

  • СI passed
  • Сode coverage >=95%
  • PR is reviewed manually again (to make sure you have 100% ready code)
  • All reviewers agreed to merge the PR
  • I've checked new feature as logged in and logged out user if needed
  • PR meets all conventions

@sonarqubecloud
Copy link
Copy Markdown

Copy link
Copy Markdown
Contributor

@Vektor19 Vektor19 left a comment

Choose a reason for hiding this comment

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

Amazing!

Copy link
Copy Markdown
Contributor

@VolodymyrShapoval VolodymyrShapoval left a comment

Choose a reason for hiding this comment

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

What I can say... You are a hero of all frontend developers! Great job!!!

@shribakb1 shribakb1 merged commit dab091c into main Dec 24, 2025
3 of 4 checks passed
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.

3 participants