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

Develop into calculators constructor base branch #985

Merged
merged 35 commits into from
Nov 27, 2024

Conversation

SleekMutt
Copy link
Contributor

Code reviewers

Summary of issue

There is no latest changes in the constructors base branch from develop branch

Summary of change

Merge changes from develop to base constructors branch

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

alexy78 and others added 30 commits October 25, 2024 14:17
* Fix scroll bug

* make scrolling smooth
* closes #931-fix-selflock-bug

* added validation in controller and test

* added request test

* before_action added

* Update app/controllers/account/users_controller.rb

Co-authored-by: Ivan Marynych <[email protected]>

* guard style return

* Update app/controllers/account/users_controller.rb

Co-authored-by: Ivan Marynych <[email protected]>

* Update users_controller.rb

* dig

* Update users_controller.rb

* Update users_controller.rb

* Update users_controller.rb

---------

Co-authored-by: Ivan Marynych <[email protected]>
…n-products

Restrict input to two decimal places in price input fields
* Created menstrual hygiene calculator. Created corresponding to it view, api controller, service and validator. Also added new corresponding action to render mhc calculator to calculators controller.

* Fixed link and rubocop offences

* Updated rubocop target version and extracted methods in pad usage service

* Deleted redundant skip

* Added tests for api controller

* Fixed some files

* Removed redundant code from controllers action

* Added smooth scroll

* Fixed localization with links

* Fixed rubocops target version

* Fixed typo

* Fixed CI

* Refactored code for a better style

* Added tests for validator

* Fixed offenses

* Added tests for each of the validate param method

* Retrieved shared example to separate file

* Added before hook for blank param

---------

Co-authored-by: Ivan Marynych <[email protected]>
Co-authored-by: loqimean <[email protected]>
Auto deploy to staging by GitHub action and Capistrano
@SleekMutt SleekMutt requested a review from loqimean November 26, 2024 18:04
Copy link

codecov bot commented Nov 27, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 85.12%. Comparing base (9130d91) to head (ffa5d1e).
Report is 36 commits behind head on calculators-constructor.

Additional details and impacted files
@@                     Coverage Diff                     @@
##           calculators-constructor     #985      +/-   ##
===========================================================
+ Coverage                    83.98%   85.12%   +1.14%     
===========================================================
  Files                           65       68       +3     
  Lines                          974     1049      +75     
===========================================================
+ Hits                           818      893      +75     
  Misses                         156      156              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@SleekMutt SleekMutt merged commit ae1497f into calculators-constructor Nov 27, 2024
7 checks passed
@SleekMutt SleekMutt deleted the update_for_constructor branch November 27, 2024 11:18
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.

8 participants