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

fix(core): ⚡ count fix for business entity get #204

Open
wants to merge 3 commits into
base: development
Choose a base branch
from
Open

fix(core): ⚡ count fix for business entity get #204

wants to merge 3 commits into from
+6 −5

Conversation

techoneel
Copy link
Member

@techoneel techoneel commented Feb 19, 2025 •
edited
Loading

Description

count fix for business entity get

Related Issues

#203

Testing

Checklist

  • I have performed a thorough self-review of my code.
  • I have added or updated relevant tests for my changes.
  • My code follows the project's style guidelines and best practices.
  • I have updated the documentation if necessary.
  • I have added or updated relevant comments in my code.
  • I have resolved any merge conflicts of my branch.

Screenshots (if applicable)

Additional Notes

Reviewers

Maintainer Notes

  • Has this change been tested in a staging environment?
  • Does this change introduce any breaking changes or deprecations?

@techoneel
fix(core): ⚡ count fix for business entity get
8a95e0d 
count fix for business entity get

Ref #203
@techoneel techoneel linked an issue Feb 19, 2025 that may be closed by this pull request
bug: business entity get total records giving wrong value #203
Open
@techoneel techoneel added the bugfix Pull Requests that fixes a bug label Feb 19, 2025
anantakumarghosh
anantakumarghosh reviewed Feb 19, 2025
View reviewed changes
Copy link
Member

@anantakumarghosh anantakumarghosh left a comment

Choose a reason for hiding this comment

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

image

on an unchanged file.
/package/utils/communication.utils.ts

@anantakumarghosh
fix(utils): 🚨 lint issues
a72d385 
exported createWhatsappMessageBody, causing lint issues

ref #203
@anantakumarghosh
Copy link
Member

anantakumarghosh commented Feb 19, 2025 •
edited
Loading

image

on an unchanged file. /package/utils/communication.utils.ts

Solved lint issues.

@anantakumarghosh
Copy link
Member

@PritamBag test this branch

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@anantakumarghosh anantakumarghosh anantakumarghosh left review comments

@PritamBag PritamBag PritamBag approved these changes

Assignees

@techoneel techoneel

Labels
bugfix Pull Requests that fixes a bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

bug: business entity get total records giving wrong value
3 participants
@techoneel @anantakumarghosh @PritamBag