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.

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

Fixed issue #434 #435

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Conversation

CooperDActor-bytes
Copy link
Contributor

fixes issue #434 of a grammatical error mistake

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
@vercel vercel bot temporarily deployed to Preview – docs January 8, 2025 06:27 Inactive
Copy link

vercel bot commented Jan 8, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Skipped Deployment
Name Status Preview Updated (UTC)
docs ⬜️ Skipped (Inspect) Jan 8, 2025 6:27am

@voidarclabs
Copy link

.-.

I'll take my pull down, no point in us both committing.

@CooperDActor-bytes
Copy link
Contributor Author

CooperDActor-bytes commented Jan 11, 2025

.-.

I'll take my pull down, no point in us both committing.

Cheers mate - fix might take a while to merge anyways since Jack (potts99) is taking a short break

@@ -49,7 +49,7 @@ export default function Tickets() {
<div className="flex flex-col">
<div className="py-2 px-6 flex flex-row items-center justify-between bg-gray-200 dark:bg-[#0A090C] border-b-[1px]">
<span className="text-sm font-bold">
You have {data.tickets.length} closed ticket
You have {data.tickets.length} closed tickets.
Copy link

Choose a reason for hiding this comment

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

This doesn't look right. Notice the logic for appending an s in the next line.

If we have multiple tickets, the string you're building will be "You have n closed tickets.s"

Copy link
Contributor Author

Choose a reason for hiding this comment

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

honestly. I really dont know typescript just thought English was English when it came to text

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants