Skip to content

feat: Used notification system in existing systems#171

Merged
2000eBe merged 8 commits into
mainfrom
feat/139-use-notification-system-for-existing-systems
May 22, 2023
Merged

feat: Used notification system in existing systems#171
2000eBe merged 8 commits into
mainfrom
feat/139-use-notification-system-for-existing-systems

Conversation

@2000eBe

@2000eBe 2000eBe commented May 19, 2023

Copy link
Copy Markdown
Contributor

PR description

Implemented methods to use Notificationsystem in existing system (affects the Regex View and Capturing View).

  • Added and deleted Capture Area
  • Started and stopped the capturing

Definition Of Done (DoD)

This PR can be squashed / merged if

  • a developer is assigned
  • the PR is NOT estimated
  • the PR is labeled
  • the PR is NOT assigned to the current sprint
  • a meaningful title has been set according to https://www.conventionalcommits.org/
  • the PR is described in detail
  • the PR links to an issue
  • the PR has been reviewed

Add additional conditions here if necessary for this PR
fix: #139

Signed-off-by: 2000eBe <erikab3cker@gmail.com>
@2000eBe 2000eBe added Phase: Construction RUP: Implementation improvement Some layout / structure / performance optimization labels May 19, 2023
@2000eBe 2000eBe self-assigned this May 19, 2023
Signed-off-by: 2000eBe <erikab3cker@gmail.com>
@2000eBe
2000eBe marked this pull request as ready for review May 19, 2023 16:27
@2000eBe
2000eBe requested review from Claiyc and tonoizer May 19, 2023 16:27
Comment thread src/components/capture-area/CaptureAreaSearchValue/CaptureAreaSearchValue.vue Outdated
Comment thread src/composables/useRunning/useRunning.ts Outdated
Comment thread src/composables/useRunning/useRunning.ts Outdated
Comment thread src/composables/useRunning/useRunning.ts Outdated
Comment thread src/composables/useRunning/useRunning.ts Outdated
Comment thread src/proc/Vigad.ts Outdated
Comment thread src/proc/Vigad.ts Outdated
Comment thread src/proc/Vigad.ts Outdated
Comment thread src/views/RegexView.vue Outdated
Comment thread src/views/RegexView.vue Outdated
Comment thread src/proc/Vigad.ts Outdated
Signed-off-by: 2000eBe <erikab3cker@gmail.com>
Comment thread src/components/capture-area/CaptureAreaSearchValue/CaptureAreaSearchValue.vue Outdated
Comment thread src/views/RegexView.vue Outdated
2000eBe and others added 5 commits May 22, 2023 21:08
Co-authored-by: Claiyc <58397976+Claiyc@users.noreply.github.com>
…SearchValue.vue

Co-authored-by: Claiyc <58397976+Claiyc@users.noreply.github.com>
Signed-off-by: 2000eBe <erikab3cker@gmail.com>

@Claiyc Claiyc left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

lgtm

@tonoizer
tonoizer self-requested a review May 22, 2023 19:45

@tonoizer tonoizer left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

lgtm

@2000eBe
2000eBe merged commit 97e5e2f into main May 22, 2023
@2000eBe
2000eBe deleted the feat/139-use-notification-system-for-existing-systems branch May 22, 2023 19:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

improvement Some layout / structure / performance optimization Phase: Construction RUP: Implementation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

use notification system for existing systems

3 participants