maxim1770 / maxim1770/app

fix парсинга Праздников Святым

Open
#33 0 comments 0 reactions 0 assignees View on GitHub

A pull request for this has already been merged.

  • #34 by @maxim1770 — merged
bug documentation refactor
Dominant language
Python
Stars
0
Forks
0
PR merge metrics
No merged PRs in 30d

Description

Баги

Заметки

  • Национальность
    Данные о национальности указаны не много где: 26 в бд, 65 на сайт c (saint-href)
    Не много данных, поэтому наверное создавать поле для этого не стоит, А МОЖЕТ И СТОИТ

  • 'переходящее' даты
    Так же данны немного, на все сайт c (saint-href) всего 11, в бд 6
    Но в бд было бы больше, если бы не баг с holiday.slug уже был задан
    Некоторые переходящие Праздники не помечены:

  • Может быть данных о переходящих датах и больше:
    Неделя 5-я Великого поста. Прп. Марии Египетской.
    2024-04-21
    Тут так же переходящая, но это мы не учли в 11, т.к нет слова 'переходящее'

  • '.' - никогда не берется в full_title т.к записана не в тегах <a>
    Пишется после окончания группы <p>
    И что интересно после Первого Праздника дня - первый праздник самый главный и большой в этот день
    Возможно иногда логические группы праздников разделяются и ';' в одном <p>
    Но скорее всего просто обозначение конца каждого full_title, часто (707 в бд) ';' попадает в full_title, и ее потом нужно убирать для holiday_title
    Точку с конца full_title нельзя удалять т.к это окончание сокращения dignity or face_sanctity
    БАГ: НО ЕСТЬ ИСКЛЮЧЕНИЕ мч. Илиодо́ра Магидского (Памфилийского) (ок. 273).; И В БД: 'мч. Илиодо́ра Магидского (Памфилийского) .' ИСПРАВИТЬ ВРУЧНУЮ ЧЕРЕЗ Update

  • Интересная ситуация
    Преставление (662), перенесение мощей прп. Максима Исповедника;
    Прп. Максима Исповедника (662);
    Подробнее читать житие на сайте, по поиску 'пе­ре­не­се­ние'

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

No source files or tests are named in the issue. Review the merged pull request #34 first, then reproduce the listed parsing and database cases; done means the affected holidays are collected correctly without reintroducing date, title, or uniqueness errors.

Written by the indexing model from the issue text.

Assessment

Tech stack
postgresql, python, sqlalchemy
Domain
backend, databases
Issue type
Bug
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
15/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.