Symfony-update, V&A-blokkade en meer - Development-iteratie #199

Nieuw jaar, nieuwe releasenotes. Tijdens de feestdagen zijn de aanwezige developers druk geweest met het updaten van verouderde Twig-templates. Verder is er een V&A-blokkade mogelijk gemaakt, is er een nieuw notificatietype en zijn er bugs gefikst.

Symfony-update

In oktober kondigden we al aan werk te gaan maken van de upgrade van Symfony 4 naar 5. Het belangrijkste obstakel daarbij was dat Symfony besloot te stoppen met ondersteuning voor php-templates.

De periode rond Kerst en oud-en-nieuw is doorgaans nogal rommelig door allerlei vrije dagen. Bovendien neemt niet elke collega dezelfde dagen vrij. Daarom werken we in die periode meestal volgens Kanban, waarbij we domweg een reeks taken klaarzetten en daar zoveel mogelijk van proberen op te lossen. Vaak gaat dat om bugs en kleine verbeteringen, deze keer kozen we ervoor om ook wat tijd daarvan te gebruiken voor het omzetten van php-templates naar twig.

We begonnen met een lijst van 323 php-templates die we nog moesten omzetten. Van die templates was van 33 stuks ook al een twig-versie, bijvoorbeeld omdat we formulierelementen zowel in php als in twig gebruikten, dus die hoeven we alleen maar te verwijderen. Uiteindelijk hebben we 199 templates weten om te zetten naar twig, waarmee we een flinke hap hebben genomen uit die lijst. Helaas betekent dat ook dat er nog 91 templates over zijn om te worden omgezet, dus we hebben nog wat werk te doen voordat we eindelijk naar Symfony 5 over kunnen.

Notificatie bij verplaatsen forumpost naar ander topic

Er is een nieuw notificatietype toegevoegd: een melding als een moderator een forumpost heeft verplaatst of verwijderd (verplaatst naar de 'Trashcan'). Er zijn drie verschillende uitkomsten, dat heeft te maken met de toegang die je hebt tot bepaalde fora:

• Als je toegang hebt tot het topic waar de reactie naartoe is verplaatst: "Een moderator heeft een forumpost van jou verplaatst naar <topic>."
• Naar een topic waar je geen toegang toe hebt: "Een moderator heeft een forumpost van jou verplaatst naar een besloten topic."
• Als een forumpost is verwijderd: "Een moderator heeft een forumpost van jou verwijderd uit <topic>."

V&A-blokkade

Het komt nog weleens voor dat de V&A-admins een gebruiker alleen voor dit onderdeel van de website willen blokkeren, bijvoorbeeld omdat de gebruiker minderjarig is, onder bewindvoering staat of een zakelijke verkoper is die zich niet (goed) aan de regels houdt. Tot op heden was het alleen mogelijk om een algehele ban te krijgen, maar nu is er ook de V&A-blokkade.

Met deze beperking kun je geen nieuwe advertenties plaatsen, geen biedingen doen en geen reactie plaatsen bij advertenties. Het is een blokkade die actieve deelname aan V&A blokkeert, waardoor het ongeveer hetzelfde is als iemand die niet is ingelogd.

Gereserveerd naar Verkocht

Het aanpassen van een gereserveerde Vraag & Aanbod-advertentie naar Verkocht was een omslachtig proces en voor sommige gebruikers was de status Verkocht helemaal niet te vinden. Deze flow is verbeterd en dit maakt het makkelijker om de status Verkocht te selecteren voor je advertentie.

Bugfixes en kleine verbeteringen

• Usabilla-feedbackwidget is verwijderd van partner- en infopagina's.
• Nieuwe reviews van Hardware Info (die effectief verwijzen naar die op Tweakers) worden niet meer weergegeven.
• Headings ([H1], enz.) werken nu ook in V&A-advertenties, evenals een inhoudsopgave ([toc]).
• Artikeltypen Overig en Column zijn beschikbaar op de review-searchlisting.
• Er zijn voorbereidingen getroffen voor web-p-uploadsupport.

Reacties (47)

[tom.cx]

Misschien een beetje flauw om hier te vernoemen maar een DarkMode zou ik toch echt fijn vinden op Tweakers. Een die bijv. meegaat met het daglicht of met een klik kan activeren zoals op nu.nl

Het nadeel van de CustomCSS is dat je het altijd aan hebt staan. Terwijl ik op de dag eigenlijk geen darkmode nodig heb.

[Reactie gewijzigd door tom.cx op 25 juli 2024 11:35]

[dev10]

Je kunt in je CustomCSS ook gebruik maken van media queries en zo dynamisch dark-mode aan en uit zetten: https://developer.mozilla...edia/prefers-color-scheme

[Gropah]

In principe is Mooie Features de plek waar feature requests gedaan kunnen worden. En daar is toevallig ook al 1 over dark mode

[tom.cx]

Die had ik gezien. Maar vond het goed om het hier ook even te benoemen.

[Reactie gewijzigd door tom.cx op 25 juli 2024 11:35]

[P_Tingen]

Er zijn meerdere tweakers die hun css beschikbaar hebben gesteld voor anderen. Zo kun je die van mij bv ook gebruiken, check https://gathering.tweaker...message/61544602#61544602 voor een paar screenshots

[_Thanatos_]

Voor de V&A, kan er iets bedacht worden voor als je meer dan 1 van het product aanbiedt, maar slechts een deel ervan verkocht hebt?

Dus stel, ik biedt 4 geheugensticks aan, en verkoop er maar 1.
* Ik zou 4 losse advertenties kunnen plaatsen, maar dan zou het handig zijn om een advertentie een X aantal keer te klonen. Dit betekent wel dat je er maar 1 tegelijk kunt verkopen.
* Maar misschien kan er ook iets komen van "1 van de 4 verkocht" als status? Dit zou handig zijn voor kopers die er bijv 2 willen hebben.

Beide opties zijn lastig met biedingen. Ik weet zo snel niet hoe je dat zou oplossen. In het tweede geval misschien dat je een prijs-per-stuk moet invullen bij je bieding, en het aantal dat je wil hebben?

[Reactie gewijzigd door _Thanatos_ op 25 juli 2024 11:35]

[MsG]

Je kan met 1 advertentie er toch gewoon meerdere verkopen? In plaats van dat er een hele speciale niche-case moet worden geprogrammeerd voor wat jij voorstelt.

[mjl]

Zo niche is dat niet, maar ik red me ook zonder speciale functionaliteit hiervoor. Als er een bieder is bied altijd even aan of deze er meer dan 1 wil (as ik die heb natuurlijk).

[Zynth]

Het is volgens mij wel niche voor de gwone tweaker.
Echter niet voor de handelaren etc. Dus deze feature bewust niet implementeren lijkt me wel goed

[_Thanatos_]

Ja dat kan ook, maar er kan ook een Mooie Feature voor gemaakt worden. Dit is tenslotte T.net

[PcDealer]

Je kan toch een verzameladvertentie maken met elk product genoemd met de aantallen en bij een wijziging dat ook in de tekst verwerken?

Dit in combinatie met een veiling is onzinnig. Je kunt wel n.o.k.t. gebruiken.

[Reactie gewijzigd door PcDealer op 25 juli 2024 11:35]

[_Thanatos_]

Dat kan zeker ja. Maar je geeft zelf al aan dat het onzinnig is icm een veiling. Vandaar dat zoiets ook tot feature verwerkt kan worden, om te voorkomen dat iedereen "maar wat doet" voor situaties als deze.

[Jeoh]

Kun je met een V&A blokkade nog wel de advertenties inzien en berichten sturen naar de (ver)kopers? Is de V&A blokkade ook zichtbaar voor andere gebruikers?

[_Thanatos_]

Ik denk dat de blokkade wel zichtbaar moet zijn, om te voorkomen dat zo iemand het via het algemene DM-berichtensysteem gaat spelen...

[PcDealer]

Inzien sowieso want:

Met deze beperking kun je geen nieuwe advertenties plaatsen, geen biedingen doen en geen reactie plaatsen bij advertenties. Het is een blokkade die actieve deelname aan V&A blokkeert, waardoor het ongeveer hetzelfde is als iemand die niet is ingelogd.

Niet ingelogde gebruikers kunnen advertenties gewoon zien. Ingelogde gebruikers uiteraard ook, maar ook gewoon gebruik maken van de direct message functie.

[ruben06]

Als Symfony developer ben ik wel super benieuwd naar welke componenten van Symfony gebruikt worden voor Tweakers.net

[HollowGamer]

Je kan zoveel van Symfony gebruiken wat je zelf wilt, wellicht zit er door de oudere code base ook nog genoeg custom stuff in?

Ik dacht wel dat Twig een beetje stilstond, word dit nog veel gebruikt? Zeker met de overstap naar Vue/React zie ik er niet echt een meerwaarde in?

[m8ZBm1Si]

Twig en Vue/React zijn niet helemaal te vergelijken. Een daarvan zit in de frontend en heeft vaak een api als backend, en Twig werkt volledig in de backend.

De php templates zijn ook templates in de backend, dus overstappen op Twig is het simpelste. Overstappen op Vue/React zou een gigantische rewrite van een groot gedeelte van de codebase betekenen.

[Gropah]

Daarbij moet je je ook afvragen of Vue/React/elk ander javascript (virtual) DOM framework wilt gebruiken. Het maakt je site super afhankelijk van javascript, wat ten koste van performance kan gaan en ik kan mij gezien de doelgroep hier voorstellen dat er relatief meer mensen zijn die noJS draaien oid.

[HollowGamer]

Ik heb hier altijd moeite mee, het is niet zo dat JavaScript je website super langzaam maakt. Elke toevoeging heeft inderdaad zijn penalty (CSS, afbeeldingen, etc.), maar als je JS goed gebruikt kan het zelfs de site ervaring verbeteren. Zo zou je hier op T.net de comments ook later kunnen inladen of meer met lazy loading kunnen doen. Uiteindelijk scheelt dat ook weer bandbreedte (voor client als ook server).

Er zijn genoeg mensen die inderdaad JS blokkeren, maar anno 2021 en met goed geoptimaliseerde websites zou het eigenlijk geen probleem meer moeten zijn.

[MsG]

Voordat we reactive frameworks op tweakers hebben zijn we denk ik wel 10 jaar verder. Volgens mij is daar een hoop technical debt gecombineerd met een historische Not Invented Here mindset. Ik kom nog overal synchrone AJAX-calls tegen als ik Tweakers bezoek, die mijn GUI locken als de verbinding even hapert. Dat is al jaren bad practice bijvoorbeeld.

[Jieve]

Mooie feature zou zijn; dat alleen mensen die bepaalde tijd op Tweakers een account hebben iets kunnen aanbieden via V&A.
Voorbeeld; mensen hebben een account gemaakt voor 1 dag om een PS5 te verkopen en daarna nooit meer terug komen op Tweakers.

[Exorcist]

Dit ja. Daar filter je ook gelijk het gros van de oplichters mee.

[mjl]

Dat doe je als Koper toch als je naar een advertentie kijkt, evenals karma, beoordelingen etc.

[Alcmaria]

Oh mooi!.. gereserveerd naar verkocht!! dat was iets waar ik altijden mee worstelde, het lukte me uiteindelijk wel maar echt supeerrrgebruiksvriendelijk was het niet :-)

dat zo'n klein ding me zo blij kan maken :-D

[Meessen]

Mooi dat gereserveerd naar verkocht makkelijk wordt, had er altijd problemen mee.

[Gropah]

edit: op totaal de verkeerde reactie gereageerd

[Reactie gewijzigd door Gropah op 25 juli 2024 11:35]

[- peter -]

Ja mooi, dat gereserveerd naar verkocht. Je moest altijd de advertentie bewerken en dan kon je daar m op verkocht zetten. Dat was wat omslachtig en inconsistent. Top hoor!

[Darkangle]

"Met deze beperking kun je geen nieuwe advertenties plaatsen, geen biedingen doen en geen reactie plaatsen bij advertenties."

Maar ze kunnen dus nog wel gewoon via de "stuur bericht naar adverteerder" direct bieden? reactie plaatsen bij een advertentie gaat zowieso in de meeste gevallen al niet; dit is door de adverteerder zelf uitgezet.

[Jay-v]

"Gereserveerd naar verkocht". YES !

Al tijden een struggle inderdaad. Zeker op mobile.

Hopelijk ook snel een verbetering in de review flow, wanneer er veel advertenties online staan bij koper of verkoper is dat nogal lastig mijn inziens.