Thanks to visit codestin.com
Credit goes to github.com

Skip to content

UI/buy plan page#483

Merged
supriya3105 merged 5 commits intodevelopfrom
UI/buy_plan_page
Jun 15, 2022
Merged

UI/buy plan page#483
supriya3105 merged 5 commits intodevelopfrom
UI/buy_plan_page

Conversation

@Shruti-Apte
Copy link
Contributor

Notion card

https://www.notion.so/saeloun/Develop-the-Plan-details-page-5dae88a09c1f48b0a9d22707161ac81a

Summary

  • Designed Buy plan details page.
  • Calculation functionality added.

Preview

Type of change

  • New feature (non-breaking change which adds functionality)

How Has This Been Tested?

Checklist:

  • I have manually tested all workflows
  • I have performed a self-review of my own code

@github-actions
Copy link

Current Code Coverage Percent of this PR:

89.18 %

Files having coverage below 100%

Impacted Files Coverage
/lib/custom_failure.rb 80.0 %
/app/controllers/users/invitations_controller.rb 86.36 %
/app/controllers/users/sessions_controller.rb 85.71 %
/app/services/invoice_payment/checkout.rb 44.0 %
/app/services/invoice_payment/pdf_generation.rb 70.97 %
/app/controllers/internal_api/v1/companies_controller.rb 95.45 %
/app/controllers/internal_api/v1/profile_controller.rb 96.88 %
/app/controllers/internal_api/v1/payment_settings_controller.rb 93.33 %
/app/controllers/internal_api/v1/wise/recipients_controller.rb 90.0 %
/app/controllers/internal_api/v1/payments/providers_controller.rb 94.74 %
/lib/benchmarking/benchmarker.rb 0.0 %

Copy link
Contributor

@onkar-saeloun onkar-saeloun left a comment

Choose a reason for hiding this comment

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

LGTM after resolving comment

@supriya3105 supriya3105 merged commit 69216d5 into develop Jun 15, 2022
@supriya3105 supriya3105 deleted the UI/buy_plan_page branch June 15, 2022 12:48
vipulnsward pushed a commit that referenced this pull request Feb 15, 2026
* Buy plan page

* Buy plan page

* resolved comments

* resolved comments

Co-authored-by: Shruti <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants