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

Skip to content

added titleize property to name for capitalizing#354

Merged
supriya3105 merged 1 commit intodevelopfrom
name-capitalisation-team-page
May 3, 2022
Merged

added titleize property to name for capitalizing#354
supriya3105 merged 1 commit intodevelopfrom
name-capitalisation-team-page

Conversation

@onkar-saeloun
Copy link
Contributor

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

Notion card

Summary

Preview

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
  • I have added automated tests for my code

@onkar-saeloun onkar-saeloun added the bug Something isn't working label May 3, 2022
@onkar-saeloun onkar-saeloun self-assigned this May 3, 2022
@supriya3105
Copy link
Contributor

@onkar-saeloun Please share the loom video for this bug

Copy link
Member

@akhilgkrishnan akhilgkrishnan left a comment

Choose a reason for hiding this comment

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

LGTM

@supriya3105 supriya3105 merged commit fda9919 into develop May 3, 2022
@supriya3105 supriya3105 deleted the name-capitalisation-team-page branch May 3, 2022 14:05
@pr-triage pr-triage bot added the PR: merged label May 3, 2022
Shruti-Apte added a commit that referenced this pull request May 6, 2022
* Basic UI for stripe connect

* Addition to basic UI

* Use react router for invoice page (#333)

* API for show invoice

* Remove show route

* Use react router for invoice page

* Address review comments

* Refactor and fix UI

* Avoid n+1 issue and use partials

* Fix rubocop failure

* Omit hash value

* Set init value as null

* added titleize property to name for capitalizing (#354)

* Invoice address spacing fixed (#355)

* Invoice address spacing fixed

* added commas

Co-authored-by: Shruti <[email protected]>

* Bump version to 0.1.9

* Heroku One Click deploy added (#357)

* Fixes unit_amount for zero_decimal_currencies

* Moves unit_amout method to invoice model using Money gem.

* Adds test cases for unit_amount.

* Fixes typo

* Perform actions project details page (#358)

* Some actions added

* Some actions added

* Prepopulating client name on generate invoice page

* Resolved comments

Co-authored-by: Shruti <[email protected]>

* Basic UI for stripe connect

* Addition to basic UI

* Updated designs

Co-authored-by: Shruti <[email protected]>
Co-authored-by: Apoorv Mishra <[email protected]>
Co-authored-by: Onkar Hasabe <[email protected]>
Co-authored-by: Akhil G Krishnan <[email protected]>
Co-authored-by: Anas Ansari <[email protected]>
vipulnsward pushed a commit that referenced this pull request Feb 15, 2026
* Basic UI for stripe connect

* Addition to basic UI

* Use react router for invoice page (#333)

* API for show invoice

* Remove show route

* Use react router for invoice page

* Address review comments

* Refactor and fix UI

* Avoid n+1 issue and use partials

* Fix rubocop failure

* Omit hash value

* Set init value as null

* added titleize property to name for capitalizing (#354)

* Invoice address spacing fixed (#355)

* Invoice address spacing fixed

* added commas

Co-authored-by: Shruti <[email protected]>

* Bump version to 0.1.9

* Heroku One Click deploy added (#357)

* Fixes unit_amount for zero_decimal_currencies

* Moves unit_amout method to invoice model using Money gem.

* Adds test cases for unit_amount.

* Fixes typo

* Perform actions project details page (#358)

* Some actions added

* Some actions added

* Prepopulating client name on generate invoice page

* Resolved comments

Co-authored-by: Shruti <[email protected]>

* Basic UI for stripe connect

* Addition to basic UI

* Updated designs

Co-authored-by: Shruti <[email protected]>
Co-authored-by: Apoorv Mishra <[email protected]>
Co-authored-by: Onkar Hasabe <[email protected]>
Co-authored-by: Akhil G Krishnan <[email protected]>
Co-authored-by: Anas Ansari <[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.

3 participants