(Feature) - Expanded decoded data collapses automatically

[Agupane]

Closes #1693 by:

• Adding shouldSafeStoreBeUpdated within UPDATE_SAFE reducer to avoid updating the safe in the case that the values didn't changed which would fire unnecessary renders of the entire app. This happens because most of the values of the safe object are arrays or nested objects that could be the same but the reference is different and it's treated as a different value
• Also on checkAndUpdateSafe avoids adding featuresEnabled if the array didn't changed

How it was before:

How it's now:

[github-actions]

CLA Assistant Lite All Contributors have signed the CLA.

[github-actions]

ESLint Summary View Full Report

Annotations are provided inline on the Files Changed tab. You can also see all annotations that were generated on the annotations page.

Type	Occurrences	Fixable
Errors	0	0
Warnings	0	0
Ignored	1	N/A

• Result: ✅ success
• Annotations: 0 total

---

^{Report generated by eslint-plus-action}

[ghost]

Travis automatic deployment:
https://pr1731--safereact.review.gnosisdev.com/rinkeby/app

[ghost]

Travis automatic deployment:
https://pr1731--safereact.review.gnosisdev.com/volta/app

[francovenica]

Checked that the "Actions" display doesn't closes with the new rendering of the table
Same with modals that opens from the table, like the one that confirms/rejects a tx or the "Send again" option
The table itself keeps re-rendering just fine, the status of the tx still update just fine, the executor tag keeps showing, the green dots keeps filling.

Looks good to me

[ghost]

Travis automatic deployment:
https://pr1731--safereact.review.gnosisdev.com/rinkeby/app

[ghost]

Travis automatic deployment:
https://pr1731--safereact.review.gnosisdev.com/volta/app

[fernandomg]

🥇 fantastic!

[fernandomg]

you're assuming this as an array of a subset of primitives, isn't it?

I mean, if you compare two objects it will give false. Maybe narrowing the type would help.

So far, instead of unknown[] you can go with something around type ComparablePrimitives = string | boolean | number and array1?: ComparablePrimitives[] | null?

not sure at all about the names, but something around that.

[nicosampler]

as I comment before, I think we should use this: https://lodash.com/docs/4.17.15#isEqual it's pretty well tested.

[Agupane]

I replaced it with lodash as @nicosampler suggested

[ghost]

Travis automatic deployment:
https://pr1731--safereact.review.gnosisdev.com/volta/app

[ghost]

Travis automatic deployment:
https://pr1731--safereact.review.gnosisdev.com/rinkeby/app

[ghost]

Travis automatic deployment:
https://pr1731--safereact.review.gnosisdev.com/rinkeby/app

[ghost]

Travis automatic deployment:
https://pr1731--safereact.review.gnosisdev.com/volta/app

[ghost]

Travis automatic deployment:
https://pr1731--safereact.review.gnosisdev.com/rinkeby/app

[ghost]

Travis automatic deployment:
https://pr1731--safereact.review.gnosisdev.com/volta/app