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

TAT-142 Add logo to site #25

Merged
merged 11 commits into from
Jul 10, 2024

Conversation

allisonrobbins
Copy link
Contributor

Add logo to homepage, nav bar, and footer

Also cleaned up some 508 technical debt:

  • A few images needed alt text
  • Add labels to some primevue components

Copy link

Copy link

Copy link

Copy link

@mehaase
Copy link
Contributor

mehaase commented Jun 28, 2024

Since this PR contains some odds and ends, I'm going to add some notes here with feedback across the entire site (even for things that were not changed in this PR). I also added a commit to tweak some of the template language.

Screenshot 2024-06-28 at 3 59 09 PM

I think you should hide the GUID and just focus on ID and name, like "M1053 - Data Backup".

Screenshot 2024-06-28 at 3 59 57 PM

@mikecarenzo just implemented support for ATT&CK citations in TIE. Can you sync up with him -- is it easy to port that code over here?

Screenshot 2024-06-28 at 4 00 50 PM

Can you enable markdown rendering for the detections text?

Screenshot 2024-06-28 at 4 02 15 PM

Can you set a max width (or column limit) for this text? In MapEx it's a col-lg-6 col-md-12, which looks nice.

Screenshot 2024-06-28 at 4 05 41 PM

Are we sure the component logic is implemented correctly? I tried setting them all to high, low, random, etc. and the top 10 list seems like it stays the same. Changing the controls definitely affects the results, but components doesn't seem to.

It also seems like it doesn't match the original calculator. Do we have unit tests for the calculator logic? Have we tested this against the existing calculator or even going all the way back to the methodology and writing new test cases from scratch?

Copy link
Contributor

@mehaase mehaase left a comment

Choose a reason for hiding this comment

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

Left some feedback above.

@mehaase
Copy link
Contributor

mehaase commented Jul 1, 2024

Can you also set the package version to 2.0.0?

Copy link

github-actions bot commented Jul 1, 2024

arobbins added 2 commits July 2, 2024 13:40
…sing spreadsheet (from updating to latest ATT&CK)
Copy link

github-actions bot commented Jul 2, 2024

Copy link

github-actions bot commented Jul 2, 2024

@allisonrobbins allisonrobbins merged commit 556853b into TAT-121-single-page-app-conversion Jul 10, 2024
2 checks passed
@allisonrobbins allisonrobbins deleted the TAT-142-create-logo branch July 10, 2024 13:47
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.

2 participants