add tests for incorrect data length in otherwise valid succession of push ops

make this a move only commit

let's add the headers in a separate diff, so it is clear that no code was changed

In D17755#404513, @emack wrote:

/modules/chronik-client/.prettierignore and /.prettierignore be removed as it's similar to .eslintignore

On another note, I'm getting the following from arc lint --everything

However, arc lint on the cashtab folder runs fine, which doesn't seem to be a local issue?

Squashed commits as requested and removed unnecessary useEffec

In D17756#404523, @alitayin wrote:

In D17756#404509, @emack wrote:

Implemented a click-outside closing mechanism for the settings window

where is this implemented? I'm not seeing event bubbling above, only the removal of the redundant useEffect.

This is a update.
Perhaps my update method wasn't quite right? It created a new sub-diff, this is the initial submission https://reviews.bitcoinabc.org/D17756?id=52941

In D17756#404509, @emack wrote:

Implemented a click-outside closing mechanism for the settings window

where is this implemented? I'm not seeing event bubbling above, only the removal of the redundant useEffect.

Please prefix your title with [cashtab] so we now in the review queue what part of the code is changed

In D17755#404513, @emack wrote:

/modules/chronik-client/.prettierignore and /.prettierignore be removed as it's similar to .eslintignore

On another note, I'm getting the following from arc lint --everything

However, arc lint on the cashtab folder runs fine, which doesn't seem to be a local issue?

/modules/chronik-client/.prettierignore and /.prettierignore be removed as it's similar to .eslintignore

Implemented a click-outside closing mechanism for the settings window

where is this implemented? I'm not seeing event bubbling above, only the removal of the redundant useEffect.

Remove useEffect