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

Add validators to uploaded files #319

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from
Open

Add validators to uploaded files #319

wants to merge 2 commits into from

Conversation

ad-m
Copy link

@ad-m ad-m commented Nov 1, 2017

Na polecenie p. Agaty Miazgi z Ministerstwa Cyfryzacji opracowałem zmiany mające na celu walidacje wprowadzanych danych celem kontroli jakości danych uwzgledniąjąć, że podstawowym wskazanym problemem w udostępnianych danych jest scalenie komórek.

zaznaczenie_1126

@ad-m
Copy link
Author

ad-m commented Nov 1, 2017

Przesłany moduł nie zawiera tłumaczenia, co jest konsekwencją wprowadzenia w projekcie nieudokumentowanych autorskich rozwiązań to zapewniających (por. #317).

@ad-m
Copy link
Author

ad-m commented Nov 2, 2017

Brak mi pomysłów dlaczego tłumaczenie nie funkcjonuje poprawnie.

@KrzysztofMadejski
Copy link
Member

KrzysztofMadejski commented Nov 2, 2017 via email

@ad-m
Copy link
Author

ad-m commented Nov 2, 2017

1/ Badam treść plików podczas edycji ich poprzez before_update w p.IResourceController, bo nie spodziewam się problemów z wydajnością, a chciałem to zrobić synchronicznie, aby było przystępnie dla użytkownika. Aby ktoś kto przygotowuje arkusz do publikacji i zapomniał o jednej komórce mógł od razu poprawić i wysłać raz jeszcze.

@KrzysztofMadejski
Copy link
Member

A jak plik jest linkowany zewnętrznie?

Obecny flow jest taki, że

  1. Wszystko jest robione asynchronicznie (qa analysis)
  2. Pliki zewnętrzne są analizowane po ich ściągnięciu lokalnie przez archivera

@ad-m
Copy link
Author

ad-m commented Nov 3, 2017

Trafna uwaga odnośnie zasobów zewnętrznych. W takim wypadku myślę, że należałoby wprowadzić drugą walidacje na danych ściąganych.

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