Shared: Improvements to alerts

[cvarley100]

Description

• Desktop: Display a test version warning on RC versions
• Desktop: Don't display update alert on RC versions

Type of change

• Enhancement (a non-breaking change which adds functionality)

How Has This Been Tested?

Tested on relevant platforms

Checklist:

Please delete items that are not relevant.

• My code follows the style guidelines for this project
• I have performed a self-review of my own code
• I have commented my code, particularly in hard-to-understand areas
• New and existing unit tests pass locally with my changes
• For changes to mobile that include native code (including React Native modules): I have verified that both iOS and Android successfully build in both Debug and Release modes
• For changes to shared: If applicable, I have verified that my changes are implemented correctly in desktop and mobile

[laumair]

Looks good.

One minor suggestion (We can cover it later though)

if (get(err, 'message') === Errors.LEDGER_CANCELLED) {
    dispatch(generateLedgerCancelledAlert(err));
} else if (get(err, 'message') === Errors.LEDGER_INVALID_INDEX) {
    dispatch(generateLedgerIncorrectIndexAlert(err));
} ... 

Instead of calling lodash.get in each if block, we can do:

const message = get(error, 'message');

if (message ===  Errors.LEDGER_CANCELLED) { }