v2.22.0 | default calendar: no symbols with customEvents

DaveChild

@sdetweil I’ve not delved far into the code but it looks to me like the symbolsForEvent function in calendar.js is getting the default symbol with the default class, but then overwriting that default with the symbol from customerEvents - but not applying the default class to it.

This might be fixed with applying this.getCalendarPropertyAsArray to ev.symbol in the line symbols[0] = ev.symbol;.

I’d do a PR myself but I have nowhere near enough experience tinkering with MagicMirror to be pushing code to a default module :). But I think that’s where the problem is, and my solution will work around that issue crudely for the moment.

sdetweil

@DaveChild cool. I’ll give that a look

DaveChild

@sdetweil I did a quick PR anyway, even if it’s not perfect, it’s better there’s a starting point for a fix than make someone repeat tracking it down to where I got to.

lmhmm

@sdetweil Yes I’ve got the same issue after upgrading to 2.22 no symbols even trying solution here. Be nice to get this resolved

sdetweil

@lmhmm Dave found it and provided a fix for next release. (April 1)

you could look at what he did and change your code,

of git pull and check the develop branch. see the instructions pinned to the top of this topic about testing fixes.

sdetweil

@DaveChild awesome. I saw all the GitHub traffic. great work

lmhmm

@sdetweil I really don’t understand the answer so will just wait for the next release. Or do I just hit pull the default calendar module? Not sure how to apply fix.

kai

Hi @DaveChild, @sdetweil,
thank you very much for your effort and your research!!!
I’ll use @DaveChild’s prefix solution until the next release; the custom icons are now displayed within the calendar.
Many, many thanks to you and all who so tirelessly develop the MM and keep it alive!
May the code be with you
-kai

sdetweil

@lmhmm you cannot just update one default module. it’s all or nothing.

mm uses the git source code mgmt system

this provides for different version of code concurrently.
mm uses the master branch as the current running code
and the develop branch to accumulate updates between releases.

if u follow the steps in the test fixes topic

https://forum.magicmirror.builders/topic/14327/testing-new-fixes-or-solving-current-problems-with-next-release-code

you can pull down the develop branch and try it out.

if you don’t know what this means and cannot try it then u have to wait. mm releases a new version every 90 days

karsten13

@sdetweil said in v2.22.0 | default calendar: no symbols with customEvents:

you can pull down the develop branch and try it out.

the fix is not on develop because it is not merged yet but it should not take a long time …

lmhmm

@karsten13 Thanks I’ll wait for the solution rather than totally mess my mirror up …

karsten13

@lmhmm

meanwhile the fix is merged on develop. You can try the develop branch or wait for official release in April.

lmhmm

@karsten13 I don’t understand your responses as they are not accompanied by any instructions so for now to avoid disaster I’ll wait thanks.

sdetweil

@lmhmm at the top of the troubleshooting section are instructions for using the develop branch to test fixes pending future delivery

https://forum.magicmirror.builders/topic/14327/testing-new-fixes-or-solving-current-problems-with-next-release-code

if you use my upgrade script to migrate versions. it will detect develop branch being active and switch back to mater

lmhmm

@sdetweil

SOLVED! Upgraded to 2.22.0 then put the fa/fas bit after symbol. It works again as before. Phweee! Just for the clueless like me!

jlaxh

Just an fyi if anyone is trying to use customEvents to alter symbols and color for a pay day event in your calendar… ask your wife not to use “$$$” as the title of the event for it. After hours of messing with the config I figured out that will alter the symbol and color for ALL events… Hopefully someone sees this and it saves them hours of frustration, especially for those of us less “coding-capable”.