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

exclude from sitemap #371

Merged
merged 5 commits into from
Oct 31, 2023
Merged

exclude from sitemap #371

merged 5 commits into from
Oct 31, 2023

Conversation

Andrea-Prague
Copy link
Collaborator

No description provided.

@met
Copy link
Collaborator

met commented Oct 20, 2023

Super, procházím tu novou sitemapu, je krásně přehlednější a kratší. Vidím tam ještš nějaké nedostatky.
Pro srovnání link na zadání

Zůstaky tam odkazy https://www.movapp.cz/kiosk*

A ještě odkazy na stránky pro generování PDF. Co se dívám, mělo by pomoct odfiltrování:

/dictionary/pdf/* , /uk/dictionary/pdf/*
/kids/pdf/*, /uk/kids/pdf/*
/kids/stories/pdf/* , /uk//kids/stories/pdf/*
/alphabet/pdf/*, /uk/alphabet/pdf/*

A ještě odfiltruj tuhle jednu adresu (nevím, jak se to tam dostalo) /pdf/.gitkeep
Snad jsem to popsal všechno.
Vyhozením toho všemo zmenšíme sitemapu na půlku.
@Andrea-Prague Zkus to prosím a já mrknu na výsledek jestli jsme ještě něco nepřehlédl.

cs: [...PDF_LINKS],
sk: ['/wiki*', '/uk/wiki*', ...PDF_LINKS],
pl: ['/wiki*', '/uk/wiki*', ...PDF_LINKS],
cs: ['/exercise/re*', '/uk/exercise/re*', ' /kiosk/*', '/uk/kiosk/*', ...PDF_LINKS],
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Namísto ' /kiosk/*', '/uk/kiosk/*' bude třeba '/kiosk*', '/uk/kiosk*'
(bez mezery na začátku a konečného lomítka), stejně tak u polské a slovenské verze:

@nataliadiak
Copy link
Contributor

nataliadiak commented Oct 20, 2023

@Andrea-Prague exclude nepodporuje regex, bohužel, viz dokumentace https://www.npmjs.com/package/next-sitemap
transform už ano.

@met
Copy link
Collaborator

met commented Oct 20, 2023

@Andrea-Prague exclude nepodporuje regex, bohužel, viz dokumentace https://www.npmjs.com/package/next-sitemap transform už ano.

aha, tak to bude důvod, proč teď nezafungovalo to PDF_LINKS a jsou ty pdf adresy ve výstupu.

Zatím těch hvězdičkových filtrů není tolik, aby vadilo je takhle ručně všechny vypsat. A strukturu URL neměníme tak často, abychom je nestíhali updatovat. Tak to zkusme s nimi, pokud to nejde líp.

@met
Copy link
Collaborator

met commented Oct 25, 2023

@Andrea-Prague Skvělé, tohle je supr.

Zbyl tam ten divný /pdf/.gitkeep - kdybys rychle přišla na to, jak se ho zbavit, zbav se ho. Kdybys nenašla snadnou cestu , můžeme to mergnou i takhle a tímhle posledním malým problémem se netrápit.

Nechápu, proč to s tečkou nebere, podle mě to máš správně, můžeš zkusit i agresivnější vzor '/pdf/*gitkeep' nebo '*gitkeep' to by nám nemělo vadit.

Copy link
Collaborator

@met met left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Díky, sitemapa teď vypadá super, můžeme mergnout.

Jen prosím tu úpravu workflows/automated-tests.yml hoďme do samostatného PR, nemíchejme ji do tohodle, kam nepatří.

This reverts commit 77d8a8d.
Copy link
Collaborator

@met met left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 let's merge it

@nataliadiak nataliadiak merged commit a6f8021 into main Oct 31, 2023
9 of 10 checks passed
@nataliadiak nataliadiak deleted the fix/sitemap-exclude branch October 31, 2023 22:21
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.

3 participants