Skip to content
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

feat: init audit-events #67

Open
wants to merge 14 commits into
base: main
Choose a base branch
from
Open

feat: init audit-events #67

wants to merge 14 commits into from

Conversation

GDS-Dominic
Copy link
Contributor

@GDS-Dominic GDS-Dominic commented Oct 26, 2023

Initial Audit Event definitions

Done:

  • Initial audit event definitions based on TxMA spreadsheet
  • Ordered Nav Bar to include audit events
  • Generated JSON schemas
  • Refactored build script to be more readable and modular
  • Set pattern for event_name in each type
  • Descriptions for each event
  • Need to minimise slots in nav bar e.g. don't duplicate for each redefinition
  • Investigate warnings for bad links
    • Fields which contain '-' are converted to '_'.
  • Range / Type needs to be set on slots
  • Examples & tests need to be created
    • Test for description being present
  • Need to upload json_schemas somewhere
  • Cross check with Splunk
  • Need input from technical writers for definitions - Raised with Sabrina
  • Metadata - Done an example Annotation
  • Speak to TxMA about mispelling
  • Look into correct URI
  • Put versioning e.g. audit/V1

@GDS-Dominic GDS-Dominic marked this pull request as ready for review November 21, 2023 14:58
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.

1 participant