TallyArbiter icon indicating copy to clipboard operation
TallyArbiter copied to clipboard

Update tallyarbiter-m5atom.ino

Open The-Sizzler opened this issue 2 years ago • 7 comments

Fixed "TALLY_EXTRA_OUTPUT" code to work properly.

The-Sizzler avatar Mar 17 '22 19:03 The-Sizzler

@MatteoGheza can you have a look at this? I don't think that we should need to have an extra set of escaped quotes to make this work.

JTF4 avatar Mar 18 '22 03:03 JTF4

That is the only way I was able to get them to work. A user in the FB group had to do the same thing for their M5StickCs to get it to work.

The-Sizzler avatar Mar 23 '22 21:03 The-Sizzler

I kinda feel like we should be keeping main.js.... it is what makes it run on desktops like mac, win, etc.

josephdadams avatar Apr 03 '22 22:04 josephdadams

@josephdadams I was trying to create 2 different pull requests but I accidentally merged the edit I made to main.js into this completely separate request. See this pull request where I fix the window size issue in main.js. This pull request should only touch "tallyarbiter-m5atom.ino" https://github.com/josephdadams/TallyArbiter/pull/408

The-Sizzler avatar Apr 04 '22 16:04 The-Sizzler

Fixed another issue with the file...

The-Sizzler avatar Apr 04 '22 16:04 The-Sizzler

@MatteoGheza is it possible that there are different versions of the M5 Sticks available - because if I remember correctly, some people did not have to switch r and g...

hrueger avatar Apr 04 '22 17:04 hrueger

This isn't for the M5 Stick, it's for the M5 Atom. I have a 3, 1 I purchased when they first got released and then 2 a few months ago. They all act the same.

The-Sizzler avatar Apr 04 '22 17:04 The-Sizzler