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

Integration der v3-ScanReports #59

Closed
Lednerb opened this issue Oct 22, 2019 · 7 comments
Closed

Integration der v3-ScanReports #59

Lednerb opened this issue Oct 22, 2019 · 7 comments
Assignees
@SniperSister @sven-ahrens
Labels
enhancement New feature or request Prio:High

Comments

@Lednerb
Copy link
Contributor

Lednerb commented Oct 22, 2019

Zum Abschluss von SIWECOS/siwecos-business-layer#133 müssen noch die zwei neuen BLAv3 Endpunkte an die Webapp angeschlossen sowie das Design entsprechend angepasst werden.

Die Dokumentation zu den beiden Endpoints befindet sich hier:

  1. SiwecosScanReport
  2. SiwecosScanReport als PDF

Für die Scanner-Score-Übersicht befindet sich die berechnete Kalkulation zur Verwendung ebenfalls in der Ausgabe.

Die reports haben jeweils als key die entsprechende URL oder MailDomain, welche gescannt wurde.

Dadurch lässt sich bspw. leicht ein Accordion-Menü generieren, bei welchem man je eine Zeile mit der URL / MailDomain und dem jeweiligen Score anzeigen kann, welche per Klick ausklappt und den gesamten Report wie in v2 anzeigt.
@Lednerb Lednerb added enhancement New feature or request Prio:High labels Oct 22, 2019
@Lednerb Lednerb mentioned this issue Oct 22, 2019
Closed
@Lednerb Lednerb mentioned this issue Nov 11, 2019
Closed
@SniperSister
Copy link
Member

Das ist hier durch, oder?

@Lednerb
Copy link
Contributor Author

Lednerb commented Nov 22, 2019

@SniperSister nein, das ist noch nicht angepasst. Es wird noch der Single-URL Scan abgerufen.

@SniperSister
Copy link
Member

In der Staging?

@SniperSister
Copy link
Member

Ah, du meinst den PDF Report, right? Ok, @sven-ahrens baust du das noch ein?

@sven-ahrens
Copy link
Collaborator

Jup!

@sven-ahrens
Copy link
Collaborator

PR dazu ist draußen: #85

Nun sollten die "neuen" PDFs runtergeladen werden!

@SniperSister
Copy link
Member

Merged, danke!

@Lednerb Lednerb closed this as completed Nov 23, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@SniperSister SniperSister

@sven-ahrens sven-ahrens

Labels
enhancement New feature or request Prio:High
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@SniperSister @Lednerb @sven-ahrens