Skip to content

Conversation

@droserasprout
Copy link
Collaborator

@droserasprout droserasprout commented Jul 26, 2021

Improvements:

  • Class that holds state of TzKT SignalR subscriptions
  • Formatted logging of index messages

Bugfixes:

  • Fix displaying help messages on known errors
  • Fix missing ctx.logger in job handlers
  • Fix incorrect job scheduler configuration

@droserasprout
Copy link
Collaborator Author

Tezos profiles indexer

@droserasprout
Copy link
Collaborator Author

@droserasprout droserasprout changed the title Fix error messages, add subscribe_by_entrypoints config option SubscriptionManager, subscribe_by_entrypoints option, bugfixes Jul 30, 2021
@droserasprout
Copy link
Collaborator Author

Logs: index name

@droserasprout droserasprout marked this pull request as ready for review August 2, 2021 09:53
@droserasprout droserasprout requested a review from m-kus August 2, 2021 09:53
contracts: Optional[List[Union[str, ContractConfig]]] = None

stateless: bool = False
subscribe_by_entrypoints: bool = False
Copy link
Member

@m-kus m-kus Aug 2, 2021

Choose a reason for hiding this comment

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

More generic approach:

subscribe: default | all | by_entrypoints | by_code (future) | other

or

event_filters:
  - contracts (default)
  - types (default)
  - entrypoints

that way we can combine default filters (types + address) with entrypoints

@droserasprout droserasprout force-pushed the feat/entrypoints-subscription branch from eea699b to 5cd04ee Compare August 3, 2021 06:47
@droserasprout droserasprout force-pushed the feat/entrypoints-subscription branch from 5cd04ee to 0006fdc Compare August 3, 2021 06:53
@droserasprout droserasprout changed the title SubscriptionManager, subscribe_by_entrypoints option, bugfixes SubscriptionManager class, bugfixes Aug 3, 2021
@droserasprout droserasprout merged commit 65b9ea3 into master Aug 3, 2021
@droserasprout droserasprout deleted the feat/entrypoints-subscription branch August 3, 2021 07:20
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.

3 participants