Audit and make optional any SUMO-specific dependencies

RESOLVED FIXED in 1.3

Status

--
major
RESOLVED FIXED
9 years ago
9 years ago

People

(Reporter: ecooper, Assigned: paulc)

Tracking

unspecified
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: sumo_only, URL)

(Reporter)

Description

9 years ago
As part of the productization of SUMO we need to go through the codebase and make optional any features/software we use that may not be present on each user's machine. 

Things like Sphinx and memcached (and minify using memcache) are good examples of what we need to look for.
(Reporter)

Updated

9 years ago
Depends on: 502245
(Reporter)

Updated

9 years ago
Depends on: 502092
(Reporter)

Updated

9 years ago
Depends on: 502248
(Reporter)

Comment 1

9 years ago
How do we want to approach Sphinx for this? I was going to create a bug, but then realized it's something we could just leave out of the package all together.
(Reporter)

Comment 2

9 years ago
(In reply to comment #1)
> How do we want to approach Sphinx for this? I was going to create a bug, but
> then realized it's something we could just leave out of the package all
> together.

Same goes for the PAGELIST plugin and tables (feature that powers the l10n status pages, most popular support articles, etc). Do we want an admin panel to add/remove pages or remove it from the package?

Comment 3

9 years ago
(In reply to comment #1)
> How do we want to approach Sphinx for this? I was going to create a bug, but
> then realized it's something we could just leave out of the package all
> together.

We already have the admin panel for enable/disable newsearch.  That panel needs CSS and linking into the main admin panel (under features???).  We also need the docs on how to install it (where's that bug gone) if users decide to use it).

Comment 4

9 years ago
(In reply to comment #2)
> (In reply to comment #1)
> > How do we want to approach Sphinx for this? I was going to create a bug, but
> > then realized it's something we could just leave out of the package all
> > together.
> 
> Same goes for the PAGELIST plugin and tables (feature that powers the l10n
> status pages, most popular support articles, etc). Do we want an admin panel to
> add/remove pages or remove it from the package?

It seems like a valuable feature, so add an admin panel.
(Reporter)

Updated

9 years ago
Depends on: 502967
(Reporter)

Updated

9 years ago
Depends on: 493059
(Reporter)

Updated

9 years ago
Depends on: 460220
(Reporter)

Updated

9 years ago
Depends on: 504603
(Reporter)

Comment 5

9 years ago
Somehow this slipped my mind, but the metrics dashboard? Is this something we will be going the Live Chat route with this? Or are we going to try to make it p10n friendly?
(Reporter)

Updated

9 years ago
Depends on: 500239
(Assignee)

Comment 6

9 years ago
I'd say the metrics dashboard isn't too important for p10n yet, or at least I think we could postpone p10n-ing it. At any rate, I can do that fairly easily, there's not much code to change (if any).

Comment 7

9 years ago
The PAGELIST feature relies on sumotools and outside data.  I'm assuming I'm not on the hook for providing those parts.
(Assignee)

Updated

9 years ago
Depends on: 510101
(Assignee)

Comment 8

9 years ago
While auditing this, I spawned bug 510101.
Looking at comments.php and tiki-view_forum.tpl - a lot of comment status functionality is hard coded.
(Assignee)

Updated

9 years ago
Depends on: 510132
(Assignee)

Comment 9

9 years ago
(In reply to comment #6)
> I'd say the metrics dashboard isn't too important for p10n yet, or at least I
> think we could postpone p10n-ing it. At any rate, I can do that fairly easily,
> there's not much code to change (if any).
Roger that. I've investigated this, and I don't see any specific Firefox code - not even Mozilla specific. So it's good to go for p12n.
(Assignee)

Updated

9 years ago
Depends on: 510278
(Assignee)

Updated

9 years ago
Depends on: 510282
(Assignee)

Updated

9 years ago
Depends on: 510324
(Assignee)

Updated

9 years ago
Depends on: 510356
(Assignee)

Updated

9 years ago
Assignee: smirkingsisyphus → paul.craciunoiu
(Assignee)

Updated

9 years ago
Depends on: 510600
(Assignee)

Updated

9 years ago
Depends on: 511302
(Assignee)

Comment 10

9 years ago
James and I walked through the code and audited all files that have changed in the past 9 months, this should take care of most of our problems.
Auditing all files changed >9 months ago requires too many work-hours. We'll find problems as they arise, but odds are they'll be minor.

I also searched the code for: firefox, mozilla, fx and made sure all those instances are not hard coded.

This should be marked fixed once all its dependencies are covered.
(Assignee)

Updated

9 years ago
No longer depends on: 502092
(Assignee)

Updated

9 years ago
No longer depends on: 502248
(Assignee)

Updated

9 years ago
No longer depends on: 510132
(Assignee)

Updated

9 years ago
No longer depends on: 510324
(Assignee)

Updated

9 years ago
Status: NEW → RESOLVED
Last Resolved: 9 years ago
Resolution: --- → FIXED
Summary: Audit and make optional any SUMO specific dependencies → Audit and make optional any SUMO-specific dependencies

Updated

9 years ago
Whiteboard: sumo_only
You need to log in before you can comment on or make changes to this bug.