Skip to content

[Feature] Block workspace message in sender worker#191

Merged
ilyamore88 merged 7 commits intomasterfrom
feature/notification-about-block-workspace
Mar 1, 2021
Merged

[Feature] Block workspace message in sender worker#191
ilyamore88 merged 7 commits intomasterfrom
feature/notification-about-block-workspace

Conversation

@ilyamore88
Copy link
Copy Markdown
Member

Adds types and logic for block workspace message to sender worker. For #185.

@ilyamore88 ilyamore88 marked this pull request as ready for review February 27, 2021 16:22
Comment thread workers/sender/src/index.ts Outdated
Comment thread workers/sender/src/index.ts Outdated
Copy link
Copy Markdown
Member

@nikmel2803 nikmel2803 left a comment

Choose a reason for hiding this comment

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

типы бы вынести

Comment thread workers/sender/src/index.ts Outdated
@ilyamore88 ilyamore88 requested a review from neSpecc March 1, 2021 10:33
@ilyamore88 ilyamore88 merged commit 81b8488 into master Mar 1, 2021
@ilyamore88 ilyamore88 deleted the feature/notification-about-block-workspace branch March 1, 2021 14:32
ilyamore88 added a commit that referenced this pull request Mar 1, 2021
* Check workspace event (#184)

* add definitions for limiter worker event types

* some code refactoring

* refactor logger creating logic

* refactor limiter worker and add second event type handler

* fix error if no workspace with provided id were found

* update docs

* fixes

* update docs

* fix linter errors

* add tests

* fix docs

* fix README.md

* upgrade eslint config codex (#187)

* [Feature] Paymaster checks subscription (#186)

* update hawk.types

* refactor paymaster worker

* fix lint warnings

* update hawk.types

* add tests for paymaster worker

* remove unused envs

* update README.md

* [Feature] Cache in grouper (#170)

* save changes

* memoize hashing

* update

* init cache

* fix eslint

* add missing await and async

* add missing async

* add clearCache method

* add type to query field, use JSON.stringify

* rewrite doc

* fix doc

* move clearCache to abstract worker

Co-authored-by: ilyamore88 <themorozik@gmail.com>

* fix(js): trim source code lines (#188)

* fix(js): trim source code lines

* Update string.ts

* Update lib/utils/string.ts

Co-authored-by: Taly <vitalik7tv@yandex.ru>

Co-authored-by: Taly <vitalik7tv@yandex.ru>

* [Feature] Block workspace message in sender worker (#191)

* fix warning

* add types for block workspace message

* add some docs

* add logic for block workspace message

* fix import and type

* fix review comments

* send messages only to admins

* Metrics optimization (#193)

* optimize push time 1s -> 10s
* unhardcode push time -> .env
* `push` -> `pushAdd`

Co-authored-by: Nikita Melnikov <nikmel2803@gmail.com>
Co-authored-by: Murod Khaydarov <murod.haydarov@gmail.com>
Co-authored-by: Peter Savchenko <specc.dev@gmail.com>
Co-authored-by: Taly <vitalik7tv@yandex.ru>
Co-authored-by: Danil Beltyukov <beltyukovdanil@gmail.com>
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.

4 participants