search

in2code-de / luxletter

Newsletter system for TYPO3
https://www.in2code.de/agentur/typo3-extensions/luxletter/
23 stars 25 forks source link

[POC] top ten links by newsletter #128

Closed rr-it closed 2 years ago

rr-it commented 2 years ago

POC for https://github.com/in2code-de/luxletter/issues/126

einpraegsam commented 2 years ago

We don't know were you are going to change something? Can you please explain more or provide before/after screenshots?

einpraegsam commented 2 years ago

Ok, while digging into this issue, I now realized that this PR is part of this PR https://github.com/in2code-de/luxletter/pull/127

This change adds new Partials to the dashboard: change

I would tend to not merge this (and the other PR). because I think we could improve the UX a bit, but have to discuss this with @lefloe before.

einpraegsam commented 2 years ago

As talked before, we like the idea behind the PR but would be happy to improve UX a bit in the future. For now, we're going to close this issue.