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

Skip to content

owner can access new client button#551

Merged
supriya3105 merged 1 commit intodevelopfrom
Bug/add_new_client_owner_access
Jul 7, 2022
Merged

owner can access new client button#551
supriya3105 merged 1 commit intodevelopfrom
Bug/add_new_client_owner_access

Conversation

@Shruti-Apte
Copy link
Contributor

@Shruti-Apte Shruti-Apte commented Jul 7, 2022

Notion card

https://www.notion.so/saeloun/Add-new-client-button-not-showing-when-no-clients-are-present-012178159d6a4672a9849a83cc0bef3f

Summary

  • Owner can now access add new client button and client details page

Preview

ownerAccess

Type of change

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)

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

github-actions bot commented Jul 7, 2022

Current Code Coverage Percent of this PR:

89.25 %

Files having coverage below 100%

Impacted Files Coverage
/lib/custom_failure.rb 80.0 %
/app/services/weekly_reminder_for_missed_entries_service.rb 31.82 %
/app/controllers/users/invitations_controller.rb 92.86 %
/app/controllers/users/sessions_controller.rb 85.71 %
/app/services/invoice_payment/checkout.rb 44.0 %
/app/services/invoice_payment/pdf_generation.rb 70.0 %
/app/controllers/internal_api/v1/payment_settings_controller.rb 93.33 %
/app/controllers/internal_api/v1/profile_controller.rb 96.88 %
/app/controllers/internal_api/v1/companies_controller.rb 95.45 %
/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 %

@supriya3105 supriya3105 merged commit 491a7f2 into develop Jul 7, 2022
@supriya3105 supriya3105 deleted the Bug/add_new_client_owner_access branch July 7, 2022 07:20
vipulnsward pushed a commit that referenced this pull request Feb 15, 2026
@Shruti-Apte
owner can access new client button (#551)
829f837 
Co-authored-by: Shruti <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@apoorv1316 apoorv1316 apoorv1316 approved these changes

@supriya3105 supriya3105 Awaiting requested review from supriya3105

Assignees

No one assigned

Labels

PR: merged

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@Shruti-Apte @apoorv1316 @supriya3105