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

[BIG-57] typo: bill success message without bill number #219

Merged

Conversation

KalliopiPliogka
Copy link
Contributor

Fixed bill message. Now, bill message is showing without the bill number.

Fixed bill message. Now, bill message is showing without the bill number.
@vercel
Copy link

vercel bot commented Aug 16, 2023

@KalliopiPliogka is attempting to deploy a commit to the Bigcapital Team on Vercel.

A member of the Team first needs to authorize it.

@abouolia
Copy link
Contributor

@KalliopiPliogka Wonderful, Could you please remove the injected number value where these keywords being used. like used in BillForm.tsx.

Removed the injected number value where the deleted keywords were used.
@KalliopiPliogka
Copy link
Contributor Author

I looked for references of these keywords in the rest of the project but i only found in BillForm.tsx as you pointed out.
So, i fixed that.
If there are any other references of these keywords let me know.

@abouolia
Copy link
Contributor

@all-contributors please add @KalliopiPliogka for bug

@abouolia abouolia merged commit 58f90a0 into bigcapitalhq:develop Aug 16, 2023
@allcontributors
Copy link
Contributor

@abouolia

I've put up a pull request to add @KalliopiPliogka! 🎉

@abouolia abouolia changed the title Update index.json [BIG-57] typo: bill success message without bill number Aug 16, 2023
@abouolia
Copy link
Contributor

abouolia commented Aug 16, 2023

@KalliopiPliogka Thanks for your contribution we're appreciate that, I added you to contributors list hope keep contribute more.

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.

2 participants