Use only one start_on date; add note that justice.cz doesn't have that date

Author: encukou

docs/operations/boards.rst

@@ -3,7 +3,8 @@ Výbory spolku
 
 Historii :term:`výborů <Výbor>` si pro vlastní potřeby zaznamenáváme do `souboru boards.toml <https://github.com/pyvec/docs.pyvec.org/blob/master/src/pyvec_docs/boards.toml>`_ a z něj se generuje i tato stránka. Kdyby něco nesedělo, ověřte `historii zdrojáku této stránky <https://github.com/pyvec/docs.pyvec.org/commits/master/docs/operations/boards.rst>`_ a funkčnost :ref:`přepisů <generate_files>`.
 
-Kanonickým zdrojem pravdy je ale `výpis na justice.cz <https://or.justice.cz/ias/ui/rejstrik-firma.vysledky?subjektId=760829&typ=UPLNY>`_.
+Kanonickým zdrojem pravdy o členech výboru je výpis ze spolkového rejstříku, viz. např. `na justice.cz <https://or.justice.cz/ias/ui/rejstrik-firma.vysledky?subjektId=760829&typ=UPLNY>`_.
+(Rejstřík ale neobsahuje datum zvolení výboru a je aktualizován s administrativím zpožděním.)
 
 
 
@@ -12,8 +13,8 @@ Kanonickým zdrojem pravdy je ale `výpis na justice.cz <https://or.justice.cz/i
 
 
 
-Nově zvolený výbor
-------------------
+Výbor od 7.5.2025
+-------------------------------------------------
 
 .. list-table::
    :header-rows: 1
@@ -37,7 +38,7 @@ Nově zvolený výbor
      * `@JakubDotPy <https://github.com/JakubDotPy>`_
      *
 
-Výbor od 1.9.2022
+Výbor od 9.4.2022
 -------------------------------------------------
 
 .. list-table::
@@ -62,7 +63,7 @@ Výbor od 1.9.2022
      * `@sairon <https://github.com/sairon>`_
      * |:moneybag:|
 
-Výbor od 4.5.2019
+Výbor od 8.4.2019
 -------------------------------------------------
 
 .. list-table::

docs/operations/boards.rst.jinja

@@ -3,7 +3,8 @@ Výbory spolku
 
 Historii :term:`výborů <Výbor>` si pro vlastní potřeby zaznamenáváme do `souboru boards.toml <https://github.com/pyvec/docs.pyvec.org/blob/master/src/pyvec_docs/boards.toml>`_ a z něj se generuje i tato stránka. Kdyby něco nesedělo, ověřte `historii zdrojáku této stránky <https://github.com/pyvec/docs.pyvec.org/commits/master/docs/operations/boards.rst>`_ a funkčnost :ref:`přepisů <generate_files>`.
 
-Kanonickým zdrojem pravdy je ale `výpis na justice.cz <https://or.justice.cz/ias/ui/rejstrik-firma.vysledky?subjektId=760829&typ=UPLNY>`_.
+Kanonickým zdrojem pravdy o členech výboru je výpis ze spolkového rejstříku, `viz justice.cz <https://or.justice.cz/ias/ui/rejstrik-firma.vysledky?subjektId=760829&typ=UPLNY>`_.
+(Rejstřík ale neobsahuje datum zvolení výboru a je aktualizován s administrativím zpožděním.)
 
 
 
@@ -12,13 +13,8 @@ Kanonickým zdrojem pravdy je ale `výpis na justice.cz <https://or.justice.cz/i
 
 
 {% for board in boards %}
-{%- if board.has_started %}
 Výbor od {{ board.start_on.strftime("%-d.%-m.%Y") }}
 -------------------------------------------------
-{%- else %}
-Nově zvolený výbor
-------------------
-{%- endif %}
 
 .. list-table::
    :header-rows: 1

src/pyvec_docs/boards.py

@@ -40,26 +40,10 @@ def is_treasurer(self) -> bool:
 
 class Board(BaseModel):
     start_on: date
-    voted_on: date
-    has_started: bool
     members: list[BoardMember]
 
     model_config = {"extra": "forbid", "frozen": True}
 
-    @classmethod
-    def create(cls, voted_on=None, start_on=None, **kwargs):
-        if start_on is None:
-            start_on = voted_on
-            has_started = False
-        else:
-            has_started = True
-        return cls(
-            voted_on=voted_on,
-            start_on=start_on,
-            has_started=has_started,
-            **kwargs,
-        )
-
     @property
     def years(self) -> tuple[int, int]:
         start_year = self.start_on.year
@@ -68,15 +52,14 @@ def years(self) -> tuple[int, int]:
 
 @cache
 def load_boards(path: Path | str = BOARDS_CONFIG_PATH) -> list[Board]:
-    """Load all boards, including inactive ones"""
     data = tomllib.loads(Path(path).read_text())
     return sorted(
-        (Board.create(**board) for board in data["board"]),
+        (Board(**board) for board in data["board"]),
         key=attrgetter('start_on'),
         reverse=True,
     )
 
 @cache
 def load_current_board(path: Path | str = BOARDS_CONFIG_PATH) -> Board:
     """Load the board that is currently in power"""
-    return next(board for board in load_boards(path) if board.has_started)
+    return next(board for board in load_boards(path))

src/pyvec_docs/boards.toml

@@ -1,17 +1,12 @@
 # The Pyvec board members in time
-# The most recent board is first, previous boards are listed in descending order
+# The current board is first, previous boards are listed in descending order
 # Each member has a `name`, `github`, and optionally a `roles` array
 #
-# "voted_on" is the day when we voted for the board. It's approximate, since
-# votes can take longer than a day.
-#
-# "start_on" should ideally be the date from justice.cz.
-# If the board doesn't officially have power yet, leave it out; "voted_on` will
-# be used as default.
+# "start_on" is the day when the board assumed power.
 
 
 [[board]]
-voted_on = "2025-05-07"
+start_on = "2025-05-07"
 
 [[board.members]]
 name = "Jakub Vysoký"
@@ -37,8 +32,7 @@ github = "JakubDotPy"
 
 
 [[board]]
-start_on = "2022-09-01"
-voted_on = "2022-04-09"
+start_on = "2022-04-09"
 
 [[board.members]]
 name = "Barbora Drbohlavová"
@@ -65,8 +59,7 @@ roles = ["treasurer"]
 
 
 [[board]]
-start_on = "2019-05-04"
-voted_on = "2019-04-08"
+start_on = "2019-04-08"
 
 [[board.members]]
 name = "Martin Bílek"
@@ -93,7 +86,6 @@ github = "whiskybar"
 
 [[board]]
 start_on = "2012-03-27"
-voted_on = "2012-03-27"  # approximation
 
 [[board.members]]
 name = "Martin Bílek"

src/pyvec_docs/ext/board_dates.py

@@ -10,7 +10,7 @@
 def board_dates(app: Sphinx, config: Config):
     board = load_boards()[0]
 
-    last_vote = board.voted_on
+    last_vote = board.start_on
     next_vote = last_vote + timedelta(days=BOARDS_MANDATE_LENGTH * 365)
 
     existing_epilog = app.config.rst_epilog or ""

src/pyvec_docs/grants.py

@@ -18,8 +18,6 @@ def remove_comments(html):
 
 def get_board_member_name(username, voted_at, boards: list[Board]):
     for board in boards:  # sorted from the most recent
-        if not board.has_started:
-            continue
         if voted_at > board.start_on:
             for member in board.members:
                 if member.github == username:

tests/test_scripts_generate_grants.py

@@ -14,29 +14,26 @@
 @pytest.fixture
 def boards():
     return [  # sorted!
-        Board.create(
+        Board(
             **{
-                # No start_on; votes don't count
-                "voted_on": date(2023, 1, 1),
+                "start_on": date(2023, 1, 1),
                 "members": [
                     {"name": "Bob", "github": "bobby"},
                 ],
             }
         ),
-        Board.create(
+        Board(
             **{
                 "start_on": date(2020, 1, 1),
-                "voted_on": date(2019, 12, 1),
                 "members": [
                     {"name": "Alice", "github": "alice"},
                     {"name": "Doubravka", "github": "doubravka"},
                 ],
             }
         ),
-        Board.create(
+        Board(
             **{
                 "start_on": date(2019, 1, 1),
-                "voted_on": date(2018, 12, 1),
                 "members": [
                     {"name": "Bob", "github": "bobby"},
                 ],
@@ -49,18 +46,6 @@ def assert_boards_sorted(boards):
     assert boards == sorted(boards)
 
 
-def assert_start_on_defaults(boards):
-    assert [b.start_on for b in boards] == [
-        date(2023, 1, 1),  # from voted_on
-        date(2020, 1, 1),  # explicit
-        date(2018, 12, 1),  # explicit
-    ]
-
-
-def assert_started_values(boards):
-    assert [b.start_on for b in boards] == [False, True, True]
-
-
 def test_to_date():
     assert to_date("2020-02-12T13:22:01Z") == date(2020, 2, 12)