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

Added definition for year 2024 #76

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

Conversation

xorelvistor
Copy link
Collaborator

We still need to validate and fill some missing data

@v6ak
Copy link
Owner

v6ak commented Oct 24, 2024

Z hlediska změn na Gitu OK, z hlediska zdrojových dat tam máme minimálně dva vadné řádky.

BTW, koukám, že v masteru nemáme tu nejnovější verzi (kde defaultně zobrazuju malou tabulku). Mám to mergnout do masteru?

@xorelvistor xorelvistor marked this pull request as ready for review October 26, 2024 18:45
@xorelvistor
Copy link
Collaborator Author

xorelvistor commented Oct 26, 2024

Z hlediska změn na Gitu OK, z hlediska zdrojových dat tam máme minimálně dva vadné řádky.

BTW, koukám, že v masteru nemáme tu nejnovější verzi (kde defaultně zobrazuju malou tabulku). Mám to mergnout do masteru?

Jo, víme o tom, už by to mělo být narovnáno.

Jo, asi to klidně mergni, uvidíme pak jaká bude zpětná vazba, takhle aspoň lidi uvidí, že jsou tam 2 režimy zobrazení.

@v6ak
Copy link
Owner

v6ak commented Oct 26, 2024

OK, hodil jsem ty úpravy do masteru.

Rebasnul jsem tuto větev na master. Build je hotový. EDIT: Zdrojovou tabulku to už zpracovalo bez problémů.

@v6ak
Copy link
Owner

v6ak commented Oct 27, 2024

Ještě jsem upravil zvýraznění řádku/sloupce, na kterém mám myš. (Týká se to jen podrobné tabulky.) Nevím proč, ale bylo to téměř neviditelné.

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