rentziass / territory_wars_report

1 stars 0 forks source link

Panic error because of Sheet Name #1

Closed elric667 closed 6 years ago

elric667 commented 6 years ago

Welcome to Territory Wars Report

Please report any issue on https://github.com/rentziass/territory_wars_report Come say hi on https://discord.gg/YGUa2Fy

Enter your guild swgoh.gg's URL: https://swgoh.gg/g/3510/0otini/ Enter your opponent swgoh.gg's URL: https://swgoh.gg/g/6012/llince-syndicte/ Processed 47/47 toon rosters for 0otini... Processed 50/50 toon rosters for ∆LLi∆NCE SYNDIC∆TE... panic: sheet name must be less than 31 characters long. It is currently '32' characters long

goroutine 1 [running]: main.writeToXLSX(0xc042070c90, 0xc042070cc0, 0xc042070cf0, 0xc0422340c0, 0xc042588240, 0x0, 0x0) /Users/francescorenzi/go/src/github.com/rentziass/territory_wars_report/main.go:204 +0x3081 main.main() /Users/francescorenzi/go/src/github.com/rentziass/territory_wars_report/main.go:146 +0x11e9

rentziass commented 6 years ago

Oh, had no idea about that limit, will look into it and thanks for reporting!

rentziass commented 6 years ago

Addressed in v0.2.1, closing.

elric667 commented 6 years ago

Thank you for addressing this error. I figured out why it was caused the Other guild has special characters in their name causing the limit to be hit.