Skip to content

Updates to Monthly Key Metrics Report#9431

Merged
zachmargolis merged 5 commits intomainfrom
margolis-monthly-key-metrics-updates
Oct 23, 2023
Merged

Updates to Monthly Key Metrics Report#9431
zachmargolis merged 5 commits intomainfrom
margolis-monthly-key-metrics-updates

Conversation

@zachmargolis
Copy link
Contributor

🛠 Summary of changes

  1. Links to new handbook article (Adds an explanation doc for the Monthly Key Metrics report GSA-TTS/identity-handbook#475)
  2. Updates recipients to include contractors (per Eng Huddle item)
screenshot
Screenshot 2023-10-23 at 10 17 01 AM

@zachmargolis zachmargolis requested a review from a team October 23, 2023 17:32
For more information on how each of these metrics are calculated,
take a look at our <a href="articles/monthly-key-metrics-explainer.html">
Monthly Key Metrics Report Explainer document</a>.
</p>
Copy link
Contributor

Choose a reason for hiding this comment

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

More "I might be missing important context, but..." content from me, but this looks like a relative URL not to our handbook site, so I wouldn't expect this link to work.

➕ to the concept of including this preamble, though!

Copy link
Contributor

Choose a reason for hiding this comment

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

Double ➕ on the preamble!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

oh whoops it is missing the handbook domain -- good catch! updated in fb92334

Copy link
Contributor

@ThatSpaceGuy ThatSpaceGuy left a comment

Choose a reason for hiding this comment

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

Looks good - nice addition of the contractors email, too.

@zachmargolis zachmargolis merged commit b5eebcd into main Oct 23, 2023
@zachmargolis zachmargolis deleted the margolis-monthly-key-metrics-updates branch October 23, 2023 19:29
@mdiarra3 mdiarra3 mentioned this pull request Oct 26, 2023
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.

3 participants