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
Mixpanel track event > track(String, Map<String, dynamic>) function have issue. Kindly correct it. Added the solution. #72
Comments
Why do you want the named parameter to be removed? |
@akshayindia the |
Hi @jaredmixpanel, Thanks for the comment. Then there is a little error to make it optional you have to put the named parameter in the {} bracket. Kindly make the changes. Looking forward to it. |
I don't want to remove the named parameter, I want to correct it, or make it optional as the code is having errors while debugging or make it non-optional. I've made changes in my project but this is in the interest of Mixpanel and organizations who want to implement Mixpanel in their Flutter application. I got this error in Mixpanel SDK - 1.4.5 |
@akshayindia I do not see the error you're describing: https://github.com/mixpanel/mixpanel-flutter/blob/main/lib/mixpanel_flutter.dart#L177 |
@akshayindia I think you should add mixpanel to a fresh project and check the code again for version 1.4.5. The mistake you are mentioning doesn't exist. If you are getting errors, you must've done some changes directly to the source in pub-cache code. It is recommended not to edit that code. mixpanel-flutter/lib/mixpanel_flutter.dart Line 173 in 48d5f7a
For future reference, confirm the error by cloning the library and checking history, match with your error and then create an issue. |
Yup, in the git history folder, it is showing correct.
Sorry about this.
Thanks for sharing the reference.
…On Wed, 11 May, 2022, 12:26 am Hamir Singh Shekhawat, < ***@***.***> wrote:
@akshayindia <https://github.com/akshayindia> I think you should add
mixpanel to a fresh project and check the code again for version 1.4.5. The
mistake you are mentioning doesn't exist. If you are getting errors, you
must've done some changes directly to the source in pub-cache code. It is
recommended not to edit that code.
For your reference please check this in git history for version 1.4.5 -
https://github.com/mixpanel/mixpanel-flutter/blob/48d5f7ad0a2ac83da31b27bc2ed0637fbd4046ca/lib/mixpanel_flutter.dart#L173
For future reference, confirm the error by cloning the library and
checking history, match with your error and then create an issue.
—
Reply to this email directly, view it on GitHub
<#72 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AIW6LVJP3GKYBNCNPZFAN7LVJKWNXANCNFSM5UIK7EMQ>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
Hi Mixpanel Team,
instead of this
void track(String eventName, {Map<String, dynamic>?} properties) {
it should be function -
void track(String eventName, Map<String, dynamic>? properties) {
I've attached the screenshot and changed it in my project. please do change the plugin as well.
The text was updated successfully, but these errors were encountered: