Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: add logging and config to collectibles #3781

Merged
merged 4 commits into from
Feb 1, 2022

Conversation

Cifko
Copy link
Contributor

@Cifko Cifko commented Jan 31, 2022

Description

Added config and logging to tari collectibles.

Motivation and Context

Unite the behavior (configs, logging) of collectibles with other apps.

How Has This Been Tested?

Manually.

@Cifko Cifko force-pushed the logging-collectibles branch 3 times, most recently from 5439069 to 85db9d4 Compare February 1, 2022 10:47
@stringhandler
Copy link
Collaborator

The build process deletes applications/tari_collectibles/web-app/build/.gitkeep so just need to exclude that from commits manually until we find a way to prevent it getting deleted

config/config.toml Outdated Show resolved Hide resolved
stringhandler
stringhandler previously approved these changes Feb 1, 2022
Copy link
Collaborator

@stringhandler stringhandler left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Happy with this, let's fix the CI and it can be merged

config/config.toml Outdated Show resolved Hide resolved
stringhandler
stringhandler previously approved these changes Feb 1, 2022
@aviator-app aviator-app bot removed the mq-failed label Feb 1, 2022
@aviator-app aviator-app bot removed the mq-failed label Feb 1, 2022
@aviator-app aviator-app bot added the mq-failed label Feb 1, 2022
@stringhandler stringhandler merged commit 96a1e4e into tari-project:development Feb 1, 2022
sdbondi added a commit to sdbondi/tari that referenced this pull request Feb 8, 2022
* development:
  refactor: reduce log level of some messages (tari-project#3804)
  ci: move Apple IDs into envs (tari-project#3797)
  feat: read asset definitions from base layer (tari-project#3802)
  fix: bump flood ban messages config (tari-project#3799)
  docs: readme fixes (tari-project#3800)
  feat(wallet_ffi)!:  add base node connectivity callback to wallet ffi (tari-project#3796)
  fix: minor fixes on collectibles (tari-project#3795)
  refactor: outbound message pipeline, threads and mempool improvements (tari-project#3792)
  feat(collectibles): add delete committee member button (tari-project#3786)
  fix(comms): minor edge-case fix to handle inbound connection while dialing (tari-project#3785)
  fix(core)!: fix potential panic for sidechain merkle root with incorrect length (tari-project#3788)
  fix(core): reduce one block behind waiting period (tari-project#3798)
  feat: add specific LibWallet error code for “Fee is greater than amount” (tari-project#3793)
  fix: coinbase output recovery bug (tari-project#3789)
  feat(base-node): add number of active sync peers metric (tari-project#3784)
  chore: script to keep .gitkeep file (tari-project#3787)
  feat: re-use scanned range proofs (tari-project#3764)
  feat: ability to compile on stable rust (tari-project#3759)
  feat: add logging and config to collectibles (tari-project#3781)
  chore: remove debug info from get-mempool-state (tari-project#3782)
@Cifko Cifko deleted the logging-collectibles branch June 23, 2022 20:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants