search

HugoFara / lwt

Learn languages by reading! A language learning app stemmed from Learning with Texts (LWT).
https://hugofara.github.io/lwt/
The Unlicense
169 stars 19 forks source link

buttons in feed wizard not working #178

Closed ProgramComputer closed 10 months ago

ProgramComputer commented 10 months ago

php-dom is required for feed wizard. some buttons did not work. how_to is not clear whether filter includes or excludes.

set feed_text as empty string

ProgramComputer commented 10 months ago

If it were only for the issue explained in the title I would merge the changes immediately, but some elements from another PR are included here so I will need you to remove them first.

Some little bugs are also present (see comments).

Also, why do think php-dom is required? Did you spot any specific bug or issue?

php-dom or php-xml is required on Ubuntu 22.

Screenshot 2024-01-02 075711
ProgramComputer commented 10 months ago

All the previous issues are fixed, let's merge!

@HugoFara I tried php-dom and php-xml. Either is fine. I checked yum and I think php-xml is available and not php-dom.