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.

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

New summary charts #4498

Merged
merged 43 commits into from
May 15, 2023
Merged

New summary charts #4498

merged 43 commits into from
May 15, 2023

Conversation

lcampbell2
Copy link
Collaborator

@lcampbell2 lcampbell2 commented Apr 26, 2023 •
edited
Loading

Extend landing page and org summaries by creating a tiered approach to measuring domain security.

  • Tier 1: Minimum Requirements. The current summaries that are displayed. These should act as a minimum first step for departments' domain security. On org details pages the hide/show button is used to view the scores including any hidden domains.
    image
  • Tier 2: Improved Posture. This tier is split into the categories displayed on the domain results page. As departments work to get rid of negative tags, they will see these charts improve.
    image
  • Tier 3: Compliance. This displays how many domains are in complete compliance with Appendix G guidance on web and email security.
    image

closes #4089, closes #4474, closes #4475

@lcampbell2
create summaries for web connections(https+hsts), ssl(TLS), spf, and …
edb2e55 
…dkim
@lcampbell2
create api queries
1dda1cc
@lcampbell2
create frontend queries for tiered summaries
80e45d0
@lcampbell2
fix ssl summary name
ee73052
@lcampbell2
update faked schema
caedfb5
@lcampbell2
create components for tiered summaries
9235dba
@lcampbell2
add new summaries to org summaries
f453fa3
@lcampbell2
make tabbed summary component
ceb9bdf
@lcampbell2
give orgs summary stats for hidden domains
d5c2590
@lcampbell2
move arrow btns to top of TieredSummaries component
d913f0d
@lcampbell2
add new org summaries to api
b436883
@lcampbell2
add chart dmarc summary
f3af39c
@lcampbell2
replace dmarcPhase summary with dmarcSummary on landing page
4c24c04
@lcampbell2
refactor to use tiered summaries on landing page and org pages
5752ddd
@lcampbell2
put tier 2 and 3 behind AB testing
a26a735
@lcampbell2
add show btn for tier one org summaries
709e40c
@lcampbell2
use SummaryGroup for rendering SummaryCards
7c7a475
@lcampbell2
remove unused org summary component
0c58c48
@lcampbell2
add dmarc summary to myTracker result
26d9b0a
@lcampbell2
put tooltip on button
d0738e8
@lcampbell2
remove unused icon
046ee5a
@lcampbell2
change tabs to accordions
7014631
@lcampbell2
add AB testing to tiered summaries
3d780b7
@lcampbell2
remove hide/show btn from A stream
4a08777
@lcampbell2
update SummaryGroup test
41fb4ac
@lcampbell2
tests
d01a99b
@lcampbell2 lcampbell2 marked this pull request as ready for review April 27, 2023 17:42
FestiveKyle
FestiveKyle approved these changes May 12, 2023
View reviewed changes
Copy link
Collaborator

@FestiveKyle FestiveKyle left a comment

Choose a reason for hiding this comment

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

Looks great!

@lcampbell2 lcampbell2 merged commit 8a2e723 into master May 15, 2023
20 checks passed
@lcampbell2 lcampbell2 deleted the new-summary-charts branch May 15, 2023 13:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@FestiveKyle FestiveKyle FestiveKyle approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Dashboard metric: Actual DMARC score Dashboard metric: Actual HTTPS score New Dashboard Metrics
2 participants
@lcampbell2 @FestiveKyle