[Cashtab] Patch proptypes issues
ClosedPublic
Actions

Authored by bytesofman on Apr 3 2024, 11:51.

Details

Reviewers

emack

Group Reviewers

Restricted Project

Commits

rABC4c8fa0a6a075: [Cashtab] Patch proptypes issues

Summary

Patch proptypes errors

SendToken initially passes '' to decimals, so accept this in proptypes
the max prop is a number for the cashtab msg field, and it works fine, so update proptypes to accept this
undefined is not a proptype, if a proptype is not isRequired, then the component accepts undefined implicitly

Test Plan

npm test

Diff Detail

Repository

rABC Bitcoin ABC

Branch

fix-proptypes

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 28284
Build 56112: Build Diff	cashtab-tests
Build 56111: arc lint + arc unit

Event Timeline

bytesofman created this revision.Apr 3 2024, 11:51

Herald added a reviewer: Restricted Project. · View Herald TranscriptApr 3 2024, 11:51

Harbormaster completed remote builds in B28284: Diff 46766.Apr 3 2024, 11:55

stop using oneoftype for cached token info

rebase

Harbormaster completed remote builds in B28285: Diff 46767.Apr 3 2024, 11:59

Harbormaster completed remote builds in B28286: Diff 46769.Apr 3 2024, 12:01

bytesofman published this revision for review.Apr 3 2024, 12:03

TokenListItem isn't liking this

This revision now requires changes to proceed.Apr 4 2024, 04:38

In D15866#359030, @emack wrote:

TokenListItem isn't liking this

I actually have this too but it's because we are both early testers

chronik-client was patched to return a string for this field while I was working on this diff

Currently, cashtab does not consider a timeFirstSeen of 'string' in the cache to be invalid, so it does not migrate the cache.

I don't think we should change proptypes to support string or num,, since it should always be num. it's expected behavior for us to see this error log.

Will clear up when I upgrade cashtab cache and we migrate again. At the moment, it's fine if this value is string, since we only use the block stamp anyway for confirmed tokens, and all new tokens will only return number from chronik-client.

emack accepted this revision.Apr 4 2024, 05:37

This revision is now accepted and ready to land.Apr 4 2024, 05:37

Closed by commit rABC4c8fa0a6a075: [Cashtab] Patch proptypes issues (authored by bytesofman). · Explain WhyApr 4 2024, 05:39

This revision was automatically updated to reflect the committed changes.

bytesofman added a commit: rABC4c8fa0a6a075: [Cashtab] Patch proptypes issues.