-
Notifications
You must be signed in to change notification settings - Fork 3
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
Initial Website Implementation #25
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #25 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 1 1
Lines 6 6
=========================================
Hits 6 6 ☔ View full report in Codecov by Sentry. |
This PR has been published to https://techniqueinferenceengine.z13.web.core.windows.net/TIE-60_initial-website-implementation/ |
4 similar comments
This PR has been published to https://techniqueinferenceengine.z13.web.core.windows.net/TIE-60_initial-website-implementation/ |
This PR has been published to https://techniqueinferenceengine.z13.web.core.windows.net/TIE-60_initial-website-implementation/ |
This PR has been published to https://techniqueinferenceengine.z13.web.core.windows.net/TIE-60_initial-website-implementation/ |
This PR has been published to https://techniqueinferenceengine.z13.web.core.windows.net/TIE-60_initial-website-implementation/ |
This PR has been published to https://techniqueinferenceengine.z13.web.core.windows.net/TIE-60_initial-website-implementation/ |
10 similar comments
This PR has been published to https://techniqueinferenceengine.z13.web.core.windows.net/TIE-60_initial-website-implementation/ |
This PR has been published to https://techniqueinferenceengine.z13.web.core.windows.net/TIE-60_initial-website-implementation/ |
This PR has been published to https://techniqueinferenceengine.z13.web.core.windows.net/TIE-60_initial-website-implementation/ |
This PR has been published to https://techniqueinferenceengine.z13.web.core.windows.net/TIE-60_initial-website-implementation/ |
This PR has been published to https://techniqueinferenceengine.z13.web.core.windows.net/TIE-60_initial-website-implementation/ |
This PR has been published to https://techniqueinferenceengine.z13.web.core.windows.net/TIE-60_initial-website-implementation/ |
This PR has been published to https://techniqueinferenceengine.z13.web.core.windows.net/TIE-60_initial-website-implementation/ |
This PR has been published to https://techniqueinferenceengine.z13.web.core.windows.net/TIE-60_initial-website-implementation/ |
This PR has been published to https://techniqueinferenceengine.z13.web.core.windows.net/TIE-60_initial-website-implementation/ |
This PR has been published to https://techniqueinferenceengine.z13.web.core.windows.net/TIE-60_initial-website-implementation/ |
@mikecarenzo Looks amazing, of course. A few requests and questions: Please remove the "research conducted by" text and logo. The way you have it in the footer makes more sense to me.
While I like the idea of deep linking to parts of the CTID website, I think it will be awkward to spell out "Center for Threat-Informed Defense" above that column of links. This might look better as a list of links in between the paragraph about the Center and the social media links. I'll let you decide how it should look. If you do keep the CTID links, I edited the links text to mirror what we have on the CTID website's navbar. (In the long run, I want us to come up with a new way of cross-linking projects that is consistent across properties and easier to maintain.) Some suggested text:
Chatting with Matt this morning and I realized that we should not put the model writeup on GitHub Wiki; we should put it on the project website. I like how Allison did the methodology page for TAT 2.0, and I could see something similar working here. Let's prioritize getting the content on the site filled in |
This PR has been published to https://techniqueinferenceengine.z13.web.core.windows.net/TIE-60_initial-website-implementation/ |
2 similar comments
This PR has been published to https://techniqueinferenceengine.z13.web.core.windows.net/TIE-60_initial-website-implementation/ |
This PR has been published to https://techniqueinferenceengine.z13.web.core.windows.net/TIE-60_initial-website-implementation/ |
5365526
to
cd63492
Compare
This PR has been published to https://techniqueinferenceengine.z13.web.core.windows.net/TIE-60_initial-website-implementation/ |
cd63492
to
70a172d
Compare
This PR has been published to https://techniqueinferenceengine.z13.web.core.windows.net/TIE-60_initial-website-implementation/ |
3 similar comments
This PR has been published to https://techniqueinferenceengine.z13.web.core.windows.net/TIE-60_initial-website-implementation/ |
This PR has been published to https://techniqueinferenceengine.z13.web.core.windows.net/TIE-60_initial-website-implementation/ |
This PR has been published to https://techniqueinferenceengine.z13.web.core.windows.net/TIE-60_initial-website-implementation/ |
dd859fe
to
8577903
Compare
8577903
to
b7b7f58
Compare
This PR has been published to https://techniqueinferenceengine.z13.web.core.windows.net/TIE-60_initial-website-implementation/ |
What Changed
Introduces the initial TIE website implementation. Major components include:
Recommenders
DataSources
DataSources
Known Limitations