Support for new notifications

[schlawg]

No description provided.

[veloce]

There a lot of indentation changes. The only change really is:

          default: {
             const url = action.notification.data['lichess.url'] as string
             if (url?.startsWith('https://lichess.org/')) {
               void openExternalBrowser(url)
             }
           }

right?

[schlawg]

correct.

sorry about the indentation i didn't have my editor hooked up to the lichobile defaults

[schlawg]

there is also probably an import openExternalBrowser at the top

[veloce]

So the default behaviour is to open the browser within the app when one taps on a notif. I make sense for "streamer goes live" and maybe "study invite", but for "tournament starting soon" it's weird since the tournaments are in the app.

Could we not open the tournament in the app instead?

[schlawg]

That's absolutely your call. If there was a change to tournament starting soon behavior, it was unintentional

[veloce]

Isn't "tournamentStartingSoon" a new notification? the app never supported it.

[schlawg]

Not sure. That one predates my changes.