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

Delete deprecated themes: BlueLagoon & Screwdriver #5374

Merged
merged 55 commits into from Jul 4, 2023

Conversation

math-GH
Copy link
Contributor

@math-GH math-GH commented May 6, 2023

Follow-up of #4807
Ref. #4295

2 month after release of 1.22.0, there was no claim to keep the themes BlueLagoon and Screwdriver.

Changes proposed in this pull request:

  • deleted the themes BlueLagoon & Screwdriver

What will happen, if a user has selected one of this themes:

  • an error message will be shown in the theme selection slider.
  • the Origine theme will be used till another theme is selected by the user

grafik

Todo for later (maybe in 1.24.0 or 1.25.0):

  • update the documentation (delete the themes in the list)

Ping @misterair, who created this 2 themes.

Pull request checklist:

  • clear commit messages
  • code manually tested

@math-GH math-GH added UI 🎨 User Interfaces Theme labels May 6, 2023
@math-GH math-GH added this to the 1.22.0 milestone May 6, 2023
@math-GH math-GH added the deprecated Features marked as deprecated or deprecated marked features will be removed label May 23, 2023
ColonelMoutarde and others added 24 commits July 4, 2023 20:56
* phpstan level 7 for feedController.php

* phpstan level 7 for feedController.php

* phpstan level 7 for feedController.php

* phpstan level 7 for feedController.php

* A few fixes

---------

Co-authored-by: Luc <sanchezluc+freshrss@gmail.com>
Co-authored-by: Alexandre Alapetite <alexandre@alapetite.fr>
* phpstan level 7 for updateController.php

* phpstan level 7 for updateController.php

* Minor array syntax

---------

Co-authored-by: Luc <sanchezluc+freshrss@gmail.com>
Co-authored-by: Alexandre Alapetite <alexandre@alapetite.fr>
* docs: language table added

* Update 05_Configuration.md

* Update 05_Configuration.md

* french docs

* Unicode quote and a few fixes
(Same search&replace aslo applied to a few other files)

---------

Co-authored-by: Alexandre Alapetite <alexandre@alapetite.fr>
#fix FreshRSS#5248

Co-authored-by: maTh <math-home@web.de>
* Add compatibility with MacOS for `paste` command
* Addition to .editorconfig
Fix regression from FreshRSS#5315
which indroduced a bug for cached feeds.
We now update the `lastSeen` property of entries to account for the fact that they are unchanged but still existing.
Co-authored-by: Luc <sanchezluc+freshrss@gmail.com>
…ocumentation (FreshRSS#5349)

* i18n

* Update sub.php

* Update app/i18n/fr/sub.php

Co-authored-by: Alexandre Alapetite <alexandre@alapetite.fr>

* fix target="_blank"

---------

Co-authored-by: Alexandre Alapetite <alexandre@alapetite.fr>
* delete 04_Changing_source_code.md

* make pot

---------

Co-authored-by: Alexandre Alapetite <alexandre@alapetite.fr>
* Typed view model classes
* Add ability to provide a typed view model class to a controller
* Use `::class` instead of string for referring to classes
* Examplified with `stats` and `javascript` controllers / views (more to do)
* Also useful for extensions (my usecase today), which did not have the ability to define own view model attributes before.

* Typo
* comments added for each step

* infos about FreshRSS added in first step

* Remove reference to Kriss and Leed from install page
I do not find that informative, and quite confusing. Moved to readme instead.

---------

Co-authored-by: Alexandre Alapetite <alexandre@alapetite.fr>
Before, the system configuration was linked to the user parameter while the
user configuration was linked to the system parameter. This was an issue when
trying to retrieve some kind of configuration value in an extension.
Now, the configurations are properly linked to their parameters.
* Fix logs pagination
Regression from FreshRSS#5269

* Add better default
…#5400)

* PHPStan Level 7 for Minz_Request

* PHPStan Level 7 for FreshRSS_Feed

* PHPStan Level 7 for Minz_Error
* Fix again updateLastSeenUnchanged
FreshRSS#5382 was not good enough to fix markAsReadUponGone and introduced a regression in `entry.lastSeen`.
New approach.
Follow-up of FreshRSS#5315

* Minor change of mind

* Fix handling of lastSeen
entry.lastSeen was not always correctly initialised, and sometimes overriden
LleanaRuv and others added 24 commits July 4, 2023 20:57
* Update admin.php

Update Spanish

* Update conf.php

Spanish update

* Fix

---------

Co-authored-by: Alexandre Alapetite <alexandre@alapetite.fr>
Co-authored-by: Luc <sanchezluc+freshrss@gmail.com>
* phpstan 9 for Search.php
phpstan 9 for Share.php

* phpstan-9 for Search.php

* Better consistency for search results

---------

Co-authored-by: Luc <sanchezluc+freshrss@gmail.com>
Co-authored-by: Alexandre Alapetite <alexandre@alapetite.fr>
* Update sub.php

Update spanish translation

* Update conf.php

Update Spanish translation

* Update gen.php

Update spanish translation

* Update index.php

Update spanish translation

* Update admin.php

update spanish translation

* Fix ignore

---------

Co-authored-by: Alexandre Alapetite <alexandre@alapetite.fr>
* phpstan-8 for category class

* Another approach to nullable
FreshRSS#5434 (comment)

---------

Co-authored-by: Luc <sanchezluc+freshrss@gmail.com>
Co-authored-by: Alexandre Alapetite <alexandre@alapetite.fr>
Link to GitHub security advisory process + PGP key
* Update dark.css to lower brightness to all icons

* Update dark.rtl.css to lower brightness to all icons

* re-add p.help .icon

* re-add p.help .icon
* Remove core extensions Google Groups and Tumblr
* Google Groups seems to have remove support for RSS/ATOM FreshRSS#2838 (see e.g. https://www.theregister.com/2021/08/16/google_groups_rss/ )
* Tumblr seems to have fixed their RSS/ATOM post-GDPR FreshRSS#1924

So for both of thems, the extensions have become irrelevant.

* Cleaning
* Clarify that maximum number to keep is per feed

Signed-off-by: Christian König <ckoenig@posteo.de>

* Append //DIRTY

Signed-off-by: Christian König <ckoenig@posteo.de>

* make fix-all

* Revert wrong whitespace

* Amend Credits.md

Signed-off-by: Christian König <ckoenig@posteo.de>

---------

Signed-off-by: Christian König <ckoenig@posteo.de>
Co-authored-by: Alexandre Alapetite <alexandre@alapetite.fr>
* Add OIDC

* Update documentation.

* Update apache conf adding IfModule

* Use IfDefine for OIDC in apache conf

* Fix non-oidc support

* Fix typing

* Use IfDefine to enable OIDC

* Add OIDC support to all dockerfiles

* Re add apache Require option

* Fixes and documentation

* A few more fixes

* A bit more doc

* Change type of environment variable

* Update readme

* Correct apache config for OIDC support.

* Fix README formatting

* Update oidc control path

* Fix oidc endpoint being cached

* A bit more review

* Simplify ExpiresActive

* Add session refresh and improve caching

* Allow more different setups

* A bit more documentation

* A bit more readme

---------

Co-authored-by: Aaron Schif <aschif@netdevgroup.com>
Co-authored-by: Alexandre Alapetite <alexandre@alapetite.fr>
Co-authored-by: maTh <math-home@web.de>
Only enable the Apache auth_openidc module when actually used
Fix FreshRSS#5460
Follow-up of FreshRSS#5351
* prependTitle()

* do not need the "damn" in the alert

* update page layout improved

* release channel

* i18n labels

* add log messages while updating

* Delete updatee.php

* Update updateController.php

* Update updateController.php

* Update updateController.php

* Update updateController.php

* add getCurrentGitBranch()

* Update updateController.php

* state2 buttons

* i18n

* loading

* Update feedback.php

* Update feedback.php

* Update feedback.php

* Update extra.js

* Apply suggestions from code review

Co-authored-by: Luc SANCHEZ <4697568+ColonelMoutarde@users.noreply.github.com>

* Update updateController.php

* Update terminology

* update button is now armed

---------

Co-authored-by: Luc SANCHEZ <4697568+ColonelMoutarde@users.noreply.github.com>
Co-authored-by: Alexandre Alapetite <alexandre@alapetite.fr>
* Allow deep link to extension configuration
Full screen

* Support slider

* Add aside_configure
Fix FreshRSS#5449 (comment)
@math-GH
Copy link
Contributor Author

math-GH commented Jul 4, 2023

@Alkarex anything left or anything that we are waiting for here?

@Alkarex
Copy link
Member

Alkarex commented Jul 4, 2023

As far as I can see, those themes are not (too) broken, so I was not in a rush to delete them. I expect to receive some complaints when they are actually deleted :-)
But we can maybe do so in edge and potentially revert if any interested user wakes up.

@Alkarex Alkarex merged commit ebf62a4 into FreshRSS:edge Jul 4, 2023
1 check passed
@math-GH math-GH deleted the delete-deprecated-themes branch July 10, 2023 18:34
@math-GH
Copy link
Contributor Author

math-GH commented Oct 8, 2023

The CSS files were not deleted with this PR by accident. I opened a new PR to delete them

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
deprecated Features marked as deprecated or deprecated marked features will be removed Theme UI 🎨 User Interfaces
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

10 participants