-
Notifications
You must be signed in to change notification settings - Fork 244
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
Feedback improvements with new interface and new methods for ease of use #349
Conversation
CHANGELOG.md
Outdated
@@ -1,3 +1,9 @@ | |||
## x.x.x | |||
* Added `CountlyFeedbacks` interface to manage feedback widgets, including NPS, Surveys, and Ratings. | |||
* `presentNPS`, `presentSurvey`, and `presentRating` methods now allow displaying specific widgets by name, ID, or tag. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it is great but mb we could add function signatures with class only not with parameter names like
presentNPS(String)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i understand why @arifBurakDemiray is suggesting that but current version also feels totally fine to me
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I will give my attitudes and good luck. You rock 🪨
- Renaming files shows lots of changes, due to which its hard to review
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
there are 2 typos i spot other than those seemed fine with me. great work!
CHANGELOG.md
Outdated
@@ -1,3 +1,9 @@ | |||
## x.x.x | |||
* Added `CountlyFeedbacks` interface to manage feedback widgets, including NPS, Surveys, and Ratings. | |||
* `presentNPS`, `presentSurvey`, and `presentRating` methods now allow displaying specific widgets by name, ID, or tag. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i understand why @arifBurakDemiray is suggesting that but current version also feels totally fine to me
No description provided.