search

chilek / lms

Lan Management System (LMS) public GIT repo
http://lms.org.pl
123 stars 133 forks source link

Zapytania bazy danych GUS REGON - brak obsługi błędów #2328

Open interduo opened 1 year ago

interduo commented 1 year ago

Czy nowa funkcja jest związana z jakimś problemem? Prosimy opisać. Ostatnio usługi API rządowe ładnie powiem "nie zawsze odpowiadają". Obecnie pobierając dane w formularzu dodawania nowego klienta nie wiemy czy po kliknięciu dane nie zostały znalezione czy po prostu nastąpił błąd w komunikacji.

Opisz swoją propozycję rozwiązania W przypadku braku odpowiedzi (po np. 2 sec) popup z komunikatem: "Błąd komunikacji z GUS". W przypadku "pustej" odpowiedzi np. dla nieistniejącego lub niepoprawnego NIPu, popup z komunikatem: "Brak danych w GUS".

Alternatywne rozwiązanie Podmiana ikonki przycisku pobierz z GUS na czerwony "X" w przypadku błędu komunikacji. Zielona "fajka" na przycisku w przypadku poprawnie pobranych danych.

chilek commented 1 year ago

To już było i nadal jest - w przypadku jakichkolwiek błędów wyświetla się alert z errorem.

interduo commented 1 year ago

Nie jest jak mówisz - zarówno na mojej instancji jak i na Twoim demo.

Procedura odwzorowania:

  1. Wejdź do modułu customeradd,
  2. Kliknij "osoba prawna/firma",
  3. uzupełnij pole NIP wartością 7141979807,
  4. kliknij przycisk "pobierz z GUS",

Obecna sytuacja: po kliknięciu przycisku pobierz danych z GUS brak błędu (w konsoli developerskiej widać zapytanie ajaxowe GET ?m=gusapi&searchtype=1&searchdata=7141979807”)

Oczekujemy: Komunikat w popupie o treści "Brak wyników w bazie GUS".

Również popup nie wyskakuje gdy zrobimy: iptables -A OUTPUT -d 194.165.48.0/24 -j DROP na instancji LMS.