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

Кампании #313

Open
biomancer opened this issue Oct 4, 2013 · 9 comments
Open

Кампании #313

biomancer opened this issue Oct 4, 2013 · 9 comments

Comments

@biomancer
Copy link
Contributor

Сейчас проект заточен под выборы Мэра Москвы, нужно обобщить и добавить работу с кампаниями.

@khodzha
Copy link
Contributor

khodzha commented Oct 16, 2013

Как нужно указывать территорию проведения кампании?

@stiff
Copy link
Contributor

stiff commented Oct 16, 2013

Я думаю, что так как выборы проводятся какой-то Избирательной комиссией (ИК), поэтому правильнее всего доделать полноценное дерово *ИКов ( http://ru.wikipedia.org/wiki/Избирательная_комиссия ), и тогда просто нужно будет указать ИК для кампании.

@khodzha
Copy link
Contributor

khodzha commented Oct 16, 2013

@stiff полноценное это всероссийское?

@stiff
Copy link
Contributor

stiff commented Oct 16, 2013

Не совсем понял вопрос, см. https://github.com/fbkinfo/rosvybory/blob/develop/app/models/uic.rb#L11 — сейчас только ТИК и УИК, а ЦИК'а или ИКС'ов нет. Базу конкретных комиссий сейчас наверно нет смысла искать, но чтобы была возможность залить при необходимости

@khodzha
Copy link
Contributor

khodzha commented Oct 17, 2013

@stiff так что делать-то? добавить цик и икс? :)

@stiff
Copy link
Contributor

stiff commented Oct 17, 2013

К Uic'у — да :) можно переименовать его в Election Commission, чтобы сразу было понятно что это не только УИК.

Но основной запрос — про кампании, которых вообще нет. И если совсем делать "на все случаи жизни", то надо переписать ооочень много: например, определять роли пользователей в конкретной кампании, а не вообще. Не уверен что есть смысл это делать, потому что подозреваю, что проект будет разворачиваться на кампанию, они практически не пересекаются: территориальное деление за несколько лет может поменяться (расширение Москвы, объединение областей, новые города), люди другие, роли, всё другое одним словом.

@stiff
Copy link
Contributor

stiff commented Oct 17, 2013

Ещё вспомнил, административное деление сейчас только двух типов сделано (адм. регион и муниципальный регион), вот это то же что и с уиком — думаю было бы полезно добавить субъекты. Я начал переделывать, но не доделал, и сейчас получается User'ы и UserApp'ы и UserCurrentRole принципиально по-разному работают, что конечно же нужно унифицировать.

Было (и осталось в User и UserApp): adm_region есть всегда, а если есть region, то это мун. регион, иначе - нужно использовать adm_region, и куча if'ов.

Стало (UserCurrentRole): region — это фактический регион (любого типа) , а в adm_region кешируется (чтобы избежать тяжёлый бой с activeadmin'ом, не помню что именно, я после нескольких экспериментов сдался) административный регион выбранного региона, если он есть upd: это я обманул, оказывается всё-таки кеш я убрал, но не очень расширяемо, из-за того как эти поля задаются в интерфейсе: https://github.com/fbkinfo/rosvybory/blob/develop/app/models/concerns/adm_region_delegation.rb

@khodzha
Copy link
Contributor

khodzha commented Dec 30, 2013

@stiff я так и не понял до конца, что делать нужно. начну пилить избирательные комиссии, как появится что-то относительно правдоподобное - позову ревьюить

@stiff
Copy link
Contributor

stiff commented Dec 30, 2013

@khodzha да, так самое правильное — если что-то никому не понятно, значит оно пока не нужно :)

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

3 participants