-
Notifications
You must be signed in to change notification settings - Fork 277
Closed
Labels
Code Quality 🔧Pertaining to code quality improvementsPertaining to code quality improvementsEnhancementNew or improved featuresNew or improved featureswontfixNon-issue or no intent for changesNon-issue or no intent for changes
Description
Presently we use NO_BLOCKCHAIN_CONNECTION as a flag to indicate that we still need to set a checksum_address. There are basically three possibilities:
- We know the checksum address.
- We don't know the checksum address, but we set one based on the verifying key (this is for a federated-only Character)
- We don't care about the checksum address, because we're just quickly init'ing a Bob or Enrico for one-time use.
In the third scenario, it probably makes sense to keep NO_BLOCKCHAIN_CONNECTION.
Reactions are currently unavailable
Metadata
Metadata
Labels
Code Quality 🔧Pertaining to code quality improvementsPertaining to code quality improvementsEnhancementNew or improved featuresNew or improved featureswontfixNon-issue or no intent for changesNon-issue or no intent for changes