feat(crux): add notifications #80

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.

Sign up for GitHub

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

Jump to bottom

Merged

balanceee merged 6 commits into develop from feat/notification

Aug 15, 2022

Contributor

balanceee commented Aug 12, 2022

No description provided.


          feat(web): add notifications

33420d5

balanceee requested a review from a team

August 12, 2022 08:02


          Merge branch 'develop' into feat/notification

4dbec9d

balanceee changed the title ~~feat(crux, crux-ui): add notifications~~ feat(crux): add notifications


          resolve lint requirements

b8915e7

balanceee force-pushed the feat/notification branch from 91d7864 to b8915e7 Compare

August 12, 2022 08:09

peterambrus3 reviewed

View reviewed changes

web/crux-ui/src/routes.ts Outdated Show resolved Hide resolved

peterambrus3 reviewed

View reviewed changes

web/crux/src/app/deploy/deploy.service.ts Outdated Show resolved Hide resolved

balanceee force-pushed the feat/notification branch from 831f5e3 to 0e00760 Compare

August 12, 2022 09:14

robot9706 requested changes

View reviewed changes

web/crux-ui/locales/en/notifications.json Show resolved Hide resolved

web/crux/src/app/agent/agent.service.ts Outdated Show resolved Hide resolved

web/crux/src/app/deploy/deploy.controller.ts Outdated Show resolved Hide resolved

web/crux/src/app/notification/notification.service.ts Outdated Show resolved Hide resolved

balanceee force-pushed the feat/notification branch 2 times, most recently from 589ab59 to e268cb0 Compare

August 12, 2022 09:55

robot9706 approved these changes

View reviewed changes

Contributor

robot9706 left a comment

😎

m8vago reviewed

View reviewed changes

web/crux-ui/locales/en/common.json Outdated Show resolved Hide resolved

m8vago reviewed

View reviewed changes

web/crux-ui/locales/en/notifications.json Outdated Show resolved Hide resolved

m8vago reviewed

View reviewed changes

web/crux-ui/locales/en/notifications.json Outdated Show resolved Hide resolved

m8vago reviewed

View reviewed changes

web/crux-ui/src/components/notifications/edit-notification-card.tsx Outdated Show resolved Hide resolved

m8vago reviewed

View reviewed changes

web/crux-ui/src/components/notifications/edit-notification-card.tsx Outdated Show resolved Hide resolved

m8vago reviewed

View reviewed changes

web/crux-ui/src/components/notifications/edit-notification-card.tsx Outdated Show resolved Hide resolved

m8vago reviewed

View reviewed changes

web/crux-ui/src/components/notifications/edit-notification-card.tsx Outdated Show resolved Hide resolved

m8vago reviewed

View reviewed changes

web/crux-ui/src/components/notifications/edit-notification-card.tsx Outdated Show resolved Hide resolved

m8vago reviewed

View reviewed changes

web/crux-ui/src/components/notifications/edit-notification-card.tsx Outdated Show resolved Hide resolved

m8vago reviewed

View reviewed changes

web/crux-ui/locales/en/notifications.json Outdated Show resolved Hide resolved

m8vago reviewed

View reviewed changes

web/crux-ui/src/components/notifications/notification-card.tsx Outdated Show resolved Hide resolved

m8vago reviewed

View reviewed changes

web/crux-ui/locales/en/common.json Outdated Show resolved Hide resolved

m8vago reviewed

View reviewed changes

web/crux-ui/src/elements/dyo-switch.tsx Outdated Show resolved Hide resolved

m8vago reviewed

View reviewed changes

web/crux-ui/src/elements/dyo-switch.tsx Outdated Show resolved Hide resolved

m8vago reviewed

View reviewed changes

web/crux-ui/src/models.ts Outdated Show resolved Hide resolved

m8vago reviewed

View reviewed changes

web/crux-ui/src/models.ts Show resolved Hide resolved

m8vago reviewed

View reviewed changes

web/crux-ui/src/pages/api/notifications/hook.ts Outdated Show resolved Hide resolved

m8vago reviewed

View reviewed changes

web/crux-ui/src/pages/notifications/[notificationId].tsx Outdated Show resolved Hide resolved

m8vago reviewed

View reviewed changes

web/crux-ui/src/routes.ts Outdated Show resolved Hide resolved

m8vago reviewed

View reviewed changes

web/crux-ui/src/server/crux/notification-service.ts Outdated Show resolved Hide resolved

m8vago reviewed

View reviewed changes

web/crux-ui/src/server/crux/notification-service.ts Outdated Show resolved Hide resolved

m8vago reviewed

View reviewed changes

web/crux-ui/src/utils.ts Outdated Show resolved Hide resolved

m8vago reviewed

View reviewed changes

web/crux/src/app/notification/notification.service.ts Outdated Show resolved Hide resolved

m8vago reviewed

View reviewed changes

web/crux/src/app/notification/notification.service.ts Outdated Show resolved Hide resolved

m8vago reviewed

View reviewed changes

web/crux/src/app/notification/notification.service.ts Outdated Show resolved Hide resolved

m8vago reviewed

View reviewed changes

web/crux/src/domain/deployment.ts Show resolved Hide resolved

m8vago reviewed

View reviewed changes

web/crux/src/domain/notification-templates.ts Outdated Show resolved Hide resolved

m8vago reviewed

View reviewed changes

web/crux/src/domain/notification-templates.ts Outdated Show resolved Hide resolved

m8vago reviewed

View reviewed changes

web/crux/src/domain/notification-templates.ts Outdated Show resolved Hide resolved


          resolve conversations

d788fae

balanceee force-pushed the feat/notification branch from e268cb0 to d788fae Compare

August 15, 2022 09:29


          Merge branch 'develop' into feat/notification

6a1f990

codecov-commenter commented Aug 15, 2022

Codecov Report

Merging #80 (6a1f990) into develop (f62a939) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           develop     #80   +/-   ##
=======================================
  Coverage     6.37%   6.37%           
=======================================
  Files           62      62           
  Lines         4391    4391           
=======================================
  Hits           280     280           
  Misses        4107    4107           
  Partials         4       4

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

m8vago reviewed

View reviewed changes

protobuf/proto/crux.proto Outdated Show resolved Hide resolved

m8vago reviewed

View reviewed changes

web/crux-ui/locales/en/notifications.json Outdated Show resolved Hide resolved

m8vago reviewed

View reviewed changes

web/crux-ui/src/components/notifications/edit-notification-card.tsx Outdated Show resolved Hide resolved

m8vago reviewed

View reviewed changes

web/crux-ui/src/components/notifications/edit-notification-card.tsx Outdated Show resolved Hide resolved

balanceee force-pushed the feat/notification branch 3 times, most recently from 8b5c91e to e773645 Compare

August 15, 2022 11:55


          resolve conversation

6afea3f

balanceee force-pushed the feat/notification branch from e773645 to 6afea3f Compare

August 15, 2022 11:56

m8vago approved these changes

View reviewed changes

balanceee merged commit 12421b1 into develop

balanceee deleted the feat/notification branch

August 15, 2022 11:59

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet