search

OpenWebconcept / plugin-pdc-base

GNU General Public License v3.0
1 stars 4 forks source link

feat: additional theme fields #22

Closed sanderdekroon closed 1 year ago

sanderdekroon commented 1 year ago

Added yoast and date_modified to Theme endpoints.

mvdhoek1 commented 1 year ago

Misschien nog handig om eencomposer run format te draaien, volgens mij gaan anders onze linters door elkaar heen lopen?

sanderdekroon commented 1 year ago

Dit heb ik net even geprobeerd, maar loop tegen een paar issues aan. php-cs-fixer is gelockt op een oudere versie(op dit moment 2.19.3) die geen PHP 8.1+ ondersteunt. De nieuwste versie is 3.23.0. Die upgrade is zo uitgevoerd, dus dat komt wel goed. Tenminste, als dat ook ok is wat jullie betreft.

Dan gaat de config uit van twee paden die ik niet heb:

    ->in(__DIR__.'/htdocs/wp-content/themes')
    ->notPath(__DIR__.'/htdocs/wp-content/themes/**/node_modules')

Ik snap ook niet direct waarom de php-cs-fixer van een plugin dingen moet fixen in een/meerdere thema('s) :).

Als laatste voert 'ie nogal wat wijzigingen door als ik composer run format draai. Zijn 60 bestanden worden gewijzigd. Ik heb ze net gecommit, maar ik hoor graag of het zo goed is of dat dat even ge-reversed moet worden.

mvdhoek1 commented 1 year ago

Laat deze maar even zitten voor nu, het bestand is gekopieerd zie ik vanuit ons thema en niet goed afgestemd op alleen de plugin. Ik zet deze op het lijstje voor vrijdag.