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

Skip to content

Organisation settings page#411

Merged
onkar-saeloun merged 68 commits intodevelopfrom
organisation-settings-page
Jun 13, 2022
Merged

Organisation settings page#411
onkar-saeloun merged 68 commits intodevelopfrom
organisation-settings-page

Conversation

@onkar-saeloun
Copy link
Contributor

@onkar-saeloun onkar-saeloun commented May 25, 2022

Notion card

https://www.notion.so/saeloun/Redesign-Organization-Settings-page-f54dbbb889424a299b3b46e37011d1e7

Summary

  • The New Org Settings page is designed.
  • Yup validations are added.
  • country and currency files are added which are reusable.
  • Logo upload, update and purge feature is integrated.

Preview

Miru._.Time.tracking.and.invoicing.-.7.June.2022.mp4

Type of change

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to
    not work as expected)
  • This change requires a documentation update

How Has This Been Tested?

  • Company name, business phone and standard rate should not be blank.
  • After selecting country, specific timezones should come in options.
  • Update button will be enabled only after changing any field.
  • Cancel should remove unsaved changes.

Checklist:

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

@github-actions
Copy link

github-actions bot commented Jun 8, 2022

Current Code Coverage Percent of this PR:

87.64 %

Files having coverage below 100%

Impacted Files Coverage
/app/controllers/users/invitations_controller.rb 86.36 %
/app/policies/payments/provider_policy.rb 85.71 %
/app/services/invoice_payment/checkout.rb 44.0 %
/app/services/invoice_payment/pdf_generation.rb 70.97 %
/app/services/report/filters.rb 87.1 %
/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/payments/providers_controller.rb 94.74 %
/lib/benchmarking/benchmarker.rb 0.0 %

@onkar-saeloun onkar-saeloun requested a review from apoorv1316 June 8, 2022 09:48
@onkar-saeloun onkar-saeloun marked this pull request as ready for review June 8, 2022 09:48
@onkar-saeloun onkar-saeloun requested a review from JijoBose June 9, 2022 06:36
Base automatically changed from profile-react-app to develop June 9, 2022 09:10
@supriya3105
Copy link
Contributor

@JijoBose Please review this PR.

@onkar-saeloun onkar-saeloun merged commit 26cfbc3 into develop Jun 13, 2022
@onkar-saeloun onkar-saeloun deleted the organisation-settings-page branch June 13, 2022 10:00
vipulnsward pushed a commit that referenced this pull request Feb 15, 2026
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