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

FreshRSS 1.14.0 #2298

Merged
merged 80 commits into from
Mar 31, 2019
Merged

FreshRSS 1.14.0 #2298

merged 80 commits into from
Mar 31, 2019

Conversation

Alkarex
Copy link
Member

@Alkarex Alkarex commented Mar 23, 2019

The next version 1.14.0 of FreshRSS is ready for testing!

This time, among the noticeable improvements, we introduce a first light support for Filter actions to auto-mark-as-read articles based on a search query. There are also two new nice themes by @patjennings, and the navigation and shortcuts have been tweaked slightly.
Under the hood, jQuery has been replaced by standard JavaScript ES6, and the client-side performance has been further improved. Ubuntu is now used as the Docker based image with a 3× speed improvement. Finally, several bugs have been fixed.

Those points are also the ones to focus on during testing.

Full changelog:

  • Features
    • Filter actions feature, to auto-mark-as-read based on a search query per feed #2275
    • Improve account change when using the unsafe automatic login #2288
  • UI
    • New themes Ansum and Mapco #2245
    • Rewrite jQuery and keyboard shortcut code as native JavaScript ES6 (except for graphs on the statistics pages) #2234
    • Batch scroll-as-read for better client-side and server-side performance #2199
    • Keyboard-shortcut navigation at end of feed or category continues to the next one #2255
    • Changed jump behaviour after marking articles as read #2206
    • More reactive auto-loading of articles #2268
  • Deployment
    • New default Docker image based on Ubuntu (~3 times faster, but ~2.5 times larger) #2205
      • Using Ubuntu 18.10 with PHP 7.2.15 and Apache 2.4.34
    • Alpine version updated to Alpine 3.9 with PHP 7.2.14 and Apache 2.4.38 #2238
  • Bug fixing
    • Fix feed option for marking modified articles as unread #2200
    • Fix API HTTP Authorization case-sensitivity issue introduced in FreshRSS 1.13.1 #2233
    • Fix breaking warning in Fever API #2239
    • Fix encoding problem in Fever API #2241
    • Fix author semi-colon prefix in Fever API #2281
    • Fix the reading of the environment variable COPY_SYSLOG_TO_STDERR #2260
    • Session fix when form login + HTTP auth are used #2286
    • Fix cli/user-info.php for accounts using a version of the database older than 1.12.0 #2291
  • CLI
    • Better validation of parameters #2046
    • New option --header to cli/user-info.php #2296
  • API
    • Supported by Readably (client for Android using Fever API)
  • I18n
  • Security
    • Reworked the CSRF token interaction with the session in some edge cases #2290
    • Remove deprecated CSP child-src instruction (was already replaced by frame-src) #2250
    • Ensure entry IDs are unique and cannot be set by feeds #2273
  • Misc.
    • Remove HHMV from Travis continuous integration #2249

Happy testing!

Alkarex and others added 30 commits December 25, 2018 01:30
Prefix jQuery variables with $
Start avoiding jQuery in easy cases
Use let/const
Mark-as-read requests are queued and sent max once per second
Which also simplifies the request
Much lighter rework: auto-mark-as-read, auto-remove-articles,
auto-load-more.
In particular, use a single onscroll event handler with a throtte.
Continue removing jQuery when possible.
Batch scroll as read + reducing jQuery
Fix EntryDAO tags warning for Fever API
Fever API encoding of special chars
* Two new themes

* update gitignore

* dirs cleaning + mobile ui improvements

* name added to credits file

* New line
* Less jQuery

Follow-up of #2199

* Even less jQuery + global view unread title fix

* Even less jQuery

* Yet even less jQuery

* Even less jQuery

* Reduce some events

* Even less jQuery

* jQuery gone from main view

+Fixed English i18n

* Fix feed folded view

* Remove Firefox 64 workaround

Remove workaround for Gecko bug 1514498 in Firefox 64, fixed in Firefox
65

* Split to extra.js

Avoid loading unneeded JavaScript code for the main view.
+ several adjustements

* Improve CSS transition fold category

* Rewrite shortcuts

Remove library. Much faster, shorter, one listener instead of many.
Control of the shortcut context.
Fix #2215

* Remove debug

* Minor syntax

* Filter out unwanted shortcut modifiers

* Menu overflow fix

* Typo

* Fix unfolding in mobile view

* Remove jQuery from category.js

* Remove jQuery from Global view
Alkarex and others added 16 commits March 9, 2019 19:59
…cker (#2282)

* Add Authorization header forwarding to the Nginx configuration for Docker

* Comment
* No old ID

#2273

* PostgreSQL insert or ignore
If an option used on cli is not recognized, the command
aborts and displays an error message.
If the typed option is similar to one of the recognized
options, a hint is displayed.

At the moment, there is a limitation on long options.
Short options are not validated at the moment.

See #2046
* Minor docker doc changes

Fix Træfik temp volume minor issue

* Warning for non x86-64

* Formatting

* Minor

* Wording
* Rework CSRF interaction with sessions

Fix #2288
Improve security in some edge cases
Maybe relevant for
#2125 (comment)

* Forgotten mime type
* CLI missing exit codes

#2291

* CLI catch for outdated databases

#2291 (comment)
* Draft of filter actions

* Travis

* Implement UI + finish logic

* Travis
* Add header to cli

Now there is a switch to display the header on user info.
While doing that, I've changed how the command is working to display
all users by default and to accept more than one user at once.
I also changed the display to make it more pleasing.

As this command displays all users by default. I wonder if we still
need the list user command.

See #2294

* Minor format
@Alkarex Alkarex added this to the 1.14.0 milestone Mar 23, 2019
aledeg
aledeg previously approved these changes Mar 24, 2019
Copy link
Member

@aledeg aledeg left a comment

Choose a reason for hiding this comment

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

I've tested the visual stuff and it looks good to me. I haven't checked the api though.

Mesa a jorn pels filtres d’accion
@Alkarex Alkarex merged commit d413f67 into master Mar 31, 2019
@aledeg
Copy link
Member

aledeg commented Mar 31, 2019

🎉

@Alkarex Alkarex mentioned this pull request Apr 3, 2019
javerous pushed a commit to javerous/FreshRSS that referenced this pull request Jan 20, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants