Skip to content

HOTFIX: Hypetrain progress event unmarshal error struct 'EventSubContribution'#109

Merged
nicklaw5 merged 1 commit intonicklaw5:mainfrom
jackmcguire1:nick/helix
Jul 22, 2021
Merged

HOTFIX: Hypetrain progress event unmarshal error struct 'EventSubContribution'#109
nicklaw5 merged 1 commit intonicklaw5:mainfrom
jackmcguire1:nick/helix

Conversation

@jackmcguire1
Copy link
Copy Markdown
Contributor

The hype train progress event fails to unmarshal into helix's
EventSubContribution struct as the 'Total' field is of type String
when it should be of type integer.

  • Change EventSubContribution.Total field to int64

{ "event": { "top_contributions": [{ "user_id": "602005942", "user_login": "miguell0lxd", "user_name": "miguell0lxd", "type": "bits", "total": 4 }, { "user_id": "616819305", "user_login": "le0n145", "user_name": "le0n145", "type": "subscription", "total": 1000 }], }

- The hype train progress event fails to unmarshal into helix's EventSubContribution struct as the 'Total' field is of type string, when it should be int
@coveralls
Copy link
Copy Markdown

coveralls commented Jul 22, 2021

Pull Request Test Coverage Report for Build 1055490174

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 96.602%

Totals Coverage Status
Change from base Build 1020667936: 0.0%
Covered Lines: 1052
Relevant Lines: 1089

💛 - Coveralls

@nicklaw5 nicklaw5 merged commit d401db2 into nicklaw5:main Jul 22, 2021
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.

4 participants