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

Add mixpanel log stream support #408

Merged
merged 1 commit into from
Dec 14, 2022

Conversation

inhumantsar
Copy link
Contributor

@inhumantsar inhumantsar commented Nov 30, 2022

🔧 Changes

  • This adds a log stream type and sink for Mixpanel

📚 References

auth0/go-auth0#133

🔬 Testing

Tests aren't passing due to the lack of LogStreamType and LogStreamSink for Mixpanel, see auth0/go-auth0#133 for these.

📝 Checklist

  • All new/changed/fixed functionality is covered by tests (or N/A)
  • I have added documentation for all new/changed functionality (or N/A)

Please let me know if I missed anything. Happy to add it.

@inhumantsar inhumantsar requested a review from a team as a code owner November 30, 2022 21:29
@sergiught sergiught force-pushed the feature/mixpanel-log-streams branch from 90bb08d to 04f4d96 Compare December 14, 2022 17:11
@sergiught sergiught changed the title feat: mixpanel log stream Add Mixpanel log stream support Dec 14, 2022
@sergiught sergiught changed the title Add Mixpanel log stream support Add mixpanel log stream support Dec 14, 2022
@sergiught sergiught changed the base branch from main to patch/ESD-25022-guardian-push-sns December 14, 2022 17:12
@sergiught sergiught deleted the branch auth0:main December 14, 2022 17:47
@sergiught sergiught closed this Dec 14, 2022
@sergiught sergiught reopened this Dec 14, 2022
@willvedd willvedd deleted the branch auth0:main December 14, 2022 17:48
@willvedd willvedd closed this Dec 14, 2022
@sergiught sergiught reopened this Dec 14, 2022
@sergiught sergiught force-pushed the feature/mixpanel-log-streams branch from 04f4d96 to f351d05 Compare December 14, 2022 17:50
@sergiught sergiught changed the base branch from patch/ESD-25022-guardian-push-sns to main December 14, 2022 17:50
@sergiught sergiught force-pushed the feature/mixpanel-log-streams branch from f351d05 to a35d780 Compare December 14, 2022 17:57
@sergiught
Copy link
Contributor

Thanks for the contribution @inhumantsar 👋🏻 I went ahead and got this updated with the go SDK release to support Mixpanel LogStreams.

@sergiught sergiught merged commit f7d4f43 into auth0:main Dec 14, 2022
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.

3 participants