-
Notifications
You must be signed in to change notification settings - Fork 46
New App Config wth dummy config for Form, Redux setup (Config, panel config, field config, localisation) setup initial commit #3242
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.
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
Closed
Closed
Changes from 23 commits
Commits
Show all changes
26 commits
Select commit
Hold shift + click to select a range
b0372ed
app config console commit
NabeelAyubee 6ee037e
Merge branch 'console' into new-app-config-console
NabeelAyubee a0dcfb4
clean up 1.0
NabeelAyubee 5b4ad14
clean up 2.0
NabeelAyubee 8066433
clean up 3.0
NabeelAyubee 287f097
clean up 4.0
NabeelAyubee 1e9e145
clean up 6.0
NabeelAyubee 89c3225
Merge branch 'console' into new-app-config-console
nabeelmd-eGov e5281f0
clearing consoles 7.0
NabeelAyubee e4ad1a9
clean up
NabeelAyubee ddce73b
clean up
NabeelAyubee 034cb02
clean up
NabeelAyubee b7dbb5b
performance: Improve commit message readability
NabeelAyubee 6812f30
integrate flows, roles, template
NabeelAyubee 7530a16
added debounce
NabeelAyubee 8096660
integrate fieldPanelPropertiesSlice
NabeelAyubee 1dc2c85
added integration with mdms
NabeelAyubee 01d410d
Layout Renderer fixes
Ramkrishna-egov 24dff66
added template integration & ui ux fix
NabeelAyubee 0e26590
integrated mdms config
NabeelAyubee 6b725cf
Dependent Field Changes (#3269)
Ramkrishna-egov 5748f60
mdms integrated flow and page
NabeelAyubee 317d116
page fix for forms
NabeelAyubee 1ea0976
update localisation
NabeelAyubee d701b5b
tranform integration
NabeelAyubee 6b765e0
Fixed Layout renderer for Template
Ramkrishna-egov File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Some comments aren't visible on the classic Files Changed page.
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
315 changes: 49 additions & 266 deletions
315
health/micro-ui/web/packages/modules/campaign-manager/src/components/AppPreview.js
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Make the field wrapper keyboard-accessible.
This
divis clickable but lacksrole,tabIndex, and keyboard handlers, triggering the a11y lints. Addrole="button",tabIndex={0}, and handle Enter/Space so keyboard users can select fields.📝 Committable suggestion
🧰 Tools
🪛 Biome (2.1.2)
[error] 35-46: Static Elements should not be interactive.
To add interactivity such as a mouse or key event listener to a static element, give the element an appropriate role value.
(lint/a11y/noStaticElementInteractions)
[error] 35-46: Enforce to have the onClick mouse event with the onKeyUp, the onKeyDown, or the onKeyPress keyboard event.
Actions triggered using mouse events should have corresponding keyboard events to account for keyboard-only navigation.
(lint/a11y/useKeyWithClickEvents)
🤖 Prompt for AI Agents