Skip to content

Replace notify -> declare flow with EDIT #11399

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
Draft
cibelius wants to merge 76 commits into
base: develop
Choose a base branch
from
Draft

Replace notify -> declare flow with EDIT #11399

cibelius wants to merge 76 commits into from

Conversation

@cibelius
Copy link
Contributor

@cibelius cibelius commented Dec 17, 2025

Description

Clearly describe what has been changed. Include relevant context or background.
Explain how the issue was fixed (if applicable) and the root cause.

Link this pull request to the GitHub issue (and optionally name the branch ocrvs-<issue #>)

Checklist

  • I have linked the correct Github issue under "Development"
  • I have tested the changes locally, and written appropriate tests
  • I have tested beyond the happy path (e.g. edge cases, failure paths)
  • I have updated the changelog with this change (if applicable)
  • I have updated the GitHub issue status accordingly

cibelius added 30 commits December 8, 2025 19:47
@cibelius
define edit action config
d92321a
@cibelius
progress with ui
b3ced82
@cibelius
add edit action menu
a6d54d3
@cibelius
progress with action menu
23d27d4
@cibelius
progress
26b4093
@cibelius
remove unused import
83810e3
@cibelius
fix lint issue
42fbcce
@cibelius
fix test
34be82f
@cibelius
fix lint issue
bd3ed9b
@cibelius
remove unused export
470fe7d
@cibelius
refactor
f164ff4
@cibelius
Merge remote-tracking branch 'origin/develop' into ocrvs-10939
f90e744 
# Conflicts:
#	packages/client/src/v2-events/hooks/useUserDetails.ts
@cibelius
remove console log
97b77c5
@cibelius
fix role
2cc3694
@cibelius
allow edit only if form has changes and no errors
5a36167
@cibelius
fix translation
d50cded
@cibelius
Merge branch 'develop' into ocrvs-10939
fccf5b5
@cibelius
fix role
c47daf1
@cibelius
dont use update action for edits
f7c9110
@cibelius
Merge branch 'develop' into ocrvs-10939
d06e37a
@cibelius
fixes
7e7b39e
@cibelius
Merge remote-tracking branch 'origin/develop' into ocrvs-10939
0435866 
# Conflicts:
#	packages/client/src/v2-events/features/workqueues/EventOverview/EventOverview.stories.tsx
@cibelius
minor fixes
f5cc2a4
@cibelius
minor refactor
dae3a25
@cibelius
add edit audit details
5323474
@cibelius
fix allowed actions logic
8c66843
@cibelius
minor fixes
f5e3efd
@cibelius
fix tests
a517d3f
@cibelius
fix test
52b1aa5
@cibelius
progress with reason input
db5531e
@cibelius
fix test
30d641b
@cibelius
remove console log
4cf6cf0
@cibelius
fixes
63a7ebf
@cibelius
Merge branch 'ocrvs-10939' into ocrvs-11225
0da3e97
@cibelius
fix test
a035348
@cibelius
fix test
b2895a6
@cibelius
Merge remote-tracking branch 'origin/develop' into ocrvs-11225
0f98f8a 
# Conflicts:
#	packages/client/src/v2-events/features/workqueues/EventOverview/components/ActionMenuStories/FieldAgentInteraction/Validated.stories.tsx
#	packages/client/src/v2-events/features/workqueues/EventOverview/components/ActionMenuStories/LocalRegistrarInteraction/Validated.stories.tsx
#	packages/client/src/v2-events/features/workqueues/EventOverview/components/ActionMenuStories/RegistrationAgentInteraction/Validated.stories.tsx
@cibelius
remove duplicate
e91fddd
@cibelius
fix tests
08d1076
@cibelius
fix tests
32bc2ae
@cibelius
fix prettier issue
c15241d
@cibelius
fix test
2064449
@cibelius
remove unnecessary Update tests
02bcaf6
@cibelius
add declare action to editAndRegister
d41fd28
@cibelius
dont export
a6c8a1e
@cibelius
fix scopes, add scopes.test.ts
525cf38
@cibelius
fix
3fba1cf
@cibelius
fix lint issue
9eb30ac
@cibelius
make delete declaration available only for created records
695afd5
@cibelius
use getAvailableActionsForEvent
c6629ef
@cibelius
return closeActionView(slug)
8381711
@cibelius
fix notify on edit
cade7bf
@cibelius
progress with replacing notified review flow with edit
50ee84f
@github-actions
Copy link

github-actions bot commented Dec 17, 2025

Oops! Looks like you forgot to update the changelog. When updating CHANGELOG.md, please consider the following:

  • Changelog is read by country implementors who might not always be familiar with all technical details of OpenCRVS. Keep language high-level, user friendly and avoid technical references to internals.
  • Answer "What's new?", "Why was the change made?" and "Why should I care?" for each change.
  • If it's a breaking change, include a migration guide answering "What do I need to do to upgrade?".

@cibelius cibelius changed the base branch from develop to ocrvs-11225 December 17, 2025 09:57
@cibelius cibelius changed the title Ocrvs 11306 Replace notify -> declare flow with EDIT Dec 17, 2025
Base automatically changed from ocrvs-11225 to develop December 17, 2025 11:15
@cibelius cibelius mentioned this pull request Dec 17, 2025
Open
@github-actions
Copy link

github-actions bot commented Jan 7, 2026

This PR has been marked with label stale Since it has been inactive for 20 days. It will automatically be closed in 10 days if no further activity occurs.

@github-actions github-actions bot added the Stale The pr is inactive label Jan 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

Stale The pr is inactive

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@cibelius