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

electoral_district_id => electoral_district_nr #6

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

vellotis
Copy link

@vellotis vellotis commented Jul 9, 2021

electoral_district_id on eksitav, electoral_district_nr on parem, sest väljendab valimisringkonna numbrit, mitte identifikaatorit.
"<KOV EHAK>.<electoral_district_nr>

Muidu võib tekkida tunne, et tegemist on VIS3 siseses identifikaatoriga.

`electoral_district_id` on eksitav, `electoral_district_nr` on parem, sest väljendab valimisringkonna numbrit, mitte identifikaatorit.
`"<KOV EHAK>.<electoral_district_nr>`
@vellotis
Copy link
Author

vellotis commented Jul 9, 2021

@PriitParmakson
Copy link
Contributor

Toetan ettepanekut, kuna on selge, et number on mõeldud identifikaatorina ja kitsam termin - praegusel juhul 'number' on täpsem ning seetõttu eelistatum. Teiselt poolt pole tegu suure erinevusega. Kuna leppisime kokku, et @svenheiberg on spetsifikatsiooni toimetaja, siis pakun, et Sven otsustab.

@svenheiberg
Copy link
Collaborator

KOV2021 jaoks on arendused tehtud nii EHS kui VIS3 jaoks. Muudatused tähendavad korduvtestimist, isegi kui skoobiks on väljade ümbernimetamine spetsifikatsioonis. Võimalik, et on kasulik täpsustada number/identifikaator vahekorda tekstis, oluline on pidada silmas, et number on käigus justnimelt identifikaatorina. Tuleb arutlusele järgmisteks valimisteks valmistudes, siis otsustame ka selle lõime saatuse.

@toomy4
Copy link
Contributor

toomy4 commented Oct 26, 2022

Siin pole arendust. Puhas dokumentatsiooni reaalsusega vastavusse viimine. Selle genereeritud failis viidatud koha peal on ringkonna number, mitte id. Senine dokumentatsioon väidab vastupidist.

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.

4 participants