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.

Sign up for GitHub

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

Jump to bottom

Refactor: Add capitalize utility function and use it in ProjectDetailsPage #917

Merged
merged 7 commits into from
Feb 23, 2025
Merged

Refactor: Add capitalize utility function and use it in ProjectDetailsPage #917

merged 7 commits into from
Feb 23, 2025

Conversation

shining-bluemoon-11
Copy link
Contributor

@shining-bluemoon-11 shining-bluemoon-11 commented Feb 23, 2025 •
edited
Loading

RESOLVE #907

  • - refactors the capitalization logic in the ProjectDetailsPage component by moving it to a separate utility function

@shining-bluemoon-11
Copy link
Contributor Author

@arkid15r done

arkid15r
arkid15r requested changes Feb 23, 2025
View reviewed changes
@shining-bluemoon-11
Copy link
Contributor Author

@arkid15r done

@arkid15r arkid15r force-pushed the feature/capitalize-function branch from f0b5f82 to 2d2d203 Compare February 23, 2025 18:33
arkid15r
arkid15r approved these changes Feb 23, 2025
View reviewed changes
Copy link
Collaborator

@arkid15r arkid15r left a comment

Choose a reason for hiding this comment

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

LGTM

@arkid15r arkid15r enabled auto-merge February 23, 2025 18:36
@arkid15r arkid15r added this pull request to the merge queue Feb 23, 2025
Merged via the queue into OWASP:main with commit df9a423 Feb 23, 2025
15 checks passed
@shining-bluemoon-11 shining-bluemoon-11 deleted the feature/capitalize-function branch February 24, 2025 03:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@arkid15r arkid15r arkid15r approved these changes

@kasya kasya Awaiting requested review from kasya kasya is a code owner

Assignees
No one assigned
Labels
frontend frontend-tests
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Refactor: Extract Capitalization Logic into Reusable Utility Function
2 participants
@shining-bluemoon-11 @arkid15r