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

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

Open
interduo opened this issue Nov 3, 2022 · 2 comments
Open

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

interduo opened this issue Nov 3, 2022 · 2 comments

Comments

@interduo
Copy link
Collaborator

interduo commented Nov 3, 2022

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
Copy link
Owner

chilek commented Nov 22, 2022

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

@chilek chilek closed this as completed Nov 22, 2022
@interduo
Copy link
Collaborator Author

interduo commented Nov 22, 2022

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.

@interduo interduo reopened this Nov 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants