-
Notifications
You must be signed in to change notification settings - Fork 33
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
PLANET-7365: Clean up Listing page code #2310
base: main
Are you sure you want to change the base?
Conversation
- Remove unnecesary code
93b8ea6
to
c10af5b
Compare
/unhold 69941b81-9b00-4862-a599-0e4d3d2db1d7
Test instance is ready 🚀🌑 oberon | admin | blocks report | CircleCI | composer-local.json ⌚ 2024.06.25 09:34:17 |
- Fix sonar cloud
/unhold f5fa74fd-1657-4c28-809a-f608af3c86f3
@@ -34,14 +34,7 @@ | |||
'width' => '300', | |||
'height' => '300', | |||
]; | |||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Noticing that this value is not being used by us, only by a couple of NROs. Do we really need to add it? What if we ask them to include into their themes instead?
https://github.com/search?q=org%3Agreenpeace%20author_share_buttons&type=code
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think @comzeradd would know better about this, especially if we need to warn the NROs about the change!
- Clean up
/unhold 735b6d81-37eb-4723-9cbc-fd5426f43ae2
@@ -24,6 +24,7 @@ | |||
type="button" | |||
data-bs-toggle="collapse" | |||
data-bs-target="#menu-{{ targetId }}" | |||
aria-label="Nav Link Button" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should be translatable
/unhold a14c00c0-ad5c-4e0b-8493-c62fc6eda8ae
Ref: https://jira.greenpeace.org/browse/PLANET-7365
Description
Remove duplicated, unused and obsolete code related to listing pages