Expose contract namespaces under contract.ns.{query, tx}.*

Open jacogr opened this issue 3 years ago • 4 comments

Jan 21 '22 13:01 jacogr

When do you plan to merge that change?=)

Feb 25 '22 00:02 xgreenx

There needs to be a full end-to-end test done with the apps UI for V1, V2 & V3 contracts with this change in. Once verified, it can be merged. (The same applies to the other PRs that are marked WIP, they look sane but have not been tested end-to-end)

Feb 25 '22 05:02 jacogr

This pull request introduces 2 alerts when merging 089ba538e5f546f2e3f7e14e57467aa4e7fc0eb1 into 756f817d93fb3214ce4f99f075b3b616cf82b63f - view on LGTM.com

new alerts:

2 for Prototype-polluting assignment

Mar 31 '22 09:03 lgtm-com[bot]

This pull request introduces 2 alerts when merging 9c2cdc8323a9498dcd5f0900addce931a3ff8bec into 6c3bd2f4085bfc1ab95f9753446c62db6d1159ec - view on LGTM.com

new alerts:

2 for Prototype-polluting assignment

Apr 09 '22 13:04 lgtm-com[bot]