Skip to content

Notification from standards updates#2072

Merged
golanglemonade merged 5 commits intomainfrom
enhance-ISS-1620
Feb 26, 2026
Merged

Notification from standards updates#2072
golanglemonade merged 5 commits intomainfrom
enhance-ISS-1620

Conversation

@adelowo
Copy link
Member

@adelowo adelowo commented Feb 25, 2026

  • Add new notifications for org admins and owners when there a standard revision is bumped up and there are changes in the controls
  • added a new controlDiff resolver
query {
    controlDiff(input: {
      standardID: "01KHCJ07BCVFAR95EEH74WR149"
      oldRevision: "v0.6.0"
      newRevision: "v0.7.0"
    }) {
      standardID
      oldRevision
      newRevision
      changes {
        refCode
        title
        diffs {
          field
          oldValue
          newValue
        }
      }
    }
  }

{
  "data": {
    "controlDiff": {
      "standardID": "01KHCJ07BCVFAR95EEH74WR149",
      "oldRevision": "v0.6.0",
      "newRevision": "v0.7.0",
      "changes": [
        {
          "refCode": "CC6.1",
          "title": "XXXX and Physical Access Controls",
          "diffs": [
            {
              "field": "title",
              "oldValue": null,
              "newValue": "XXXX and Physical Access Controls"
            },
            {
              "field": "description",
              "oldValue": "The entity implements logical access security",
              "newValue": "XX Logical and Physical Access Controls"
            }
          ]
        }
      ]
    }
  },
  "extensions": {
    "auth": {
      "authentication_type": "pat",
      "authorized_organization": [
        "01KJASG9V9JAVHYSZ51FYV6Q6V"
      ]
    },
    "server_latency": "86.453708ms",
    "trace_id": "JWEbYihLwYIOCdpqKEBqcDcwPUBySrWc"
  }
}

@github-actions github-actions bot added the cli label Feb 25, 2026
@adelowo adelowo marked this pull request as ready for review February 25, 2026 21:47
@adelowo adelowo requested a review from a team as a code owner February 25, 2026 21:47
return ErrEntityIDNotFound
}

allowCtx := privacy.DecisionContext(ctx.Context, privacy.Allow)
Copy link
Member

Choose a reason for hiding this comment

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

Shouldn't need the allowCtx - the ctx is the same as the updater, right?

@sonarqubecloud
Copy link

@golanglemonade golanglemonade merged commit 76400f6 into main Feb 26, 2026
20 checks passed
@golanglemonade golanglemonade deleted the enhance-ISS-1620 branch February 26, 2026 15:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants