-
Notifications
You must be signed in to change notification settings - Fork 219
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
stringhandler
merged 4 commits into
tari-project:development
from
Cifko:logging-collectibles
Feb 1, 2022
Merged
feat: add logging and config to collectibles #3781
stringhandler
merged 4 commits into
tari-project:development
from
Cifko:logging-collectibles
Feb 1, 2022
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Cifko
force-pushed
the
logging-collectibles
branch
3 times, most recently
from
February 1, 2022 10:47
5439069
to
85db9d4
Compare
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 |
stringhandler
previously approved these changes
Feb 1, 2022
There was a problem hiding this 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
Cifko
force-pushed
the
logging-collectibles
branch
2 times, most recently
from
February 1, 2022 14:03
6ec96ac
to
5f3799c
Compare
stringhandler
previously approved these changes
Feb 1, 2022
Cifko
force-pushed
the
logging-collectibles
branch
from
February 1, 2022 14:09
987601a
to
9696c30
Compare
stringhandler
approved these changes
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)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.