Page MenuHomePhabricator

[EPIC] Move Nearby code to more suitable extension
Closed, DuplicatePublic

Description

This would require moving some of our code and putting it into a new extension.

This would probably be a useful for code sharing between Flow, MobileFrontend and Geodata so is probably a good idea.

Options:

  • move it into GeoData
    • GeoData is a data-only extension and should remain so
  • Kartographer
    • Makes it easier to integrate with maps.
    • Makes this extension depend on GeoData extension
    • Makes it harder to use this functionality without setting up all the maps stuff
  • New Extension
    • More extensions to maintain.
    • Isolated from other code
    • new extension could take a configurable API endpoint a la Popups meaning it would have a soft dependency on Geodata and its API endpoint.

Version: unspecified
Severity: normal

Related Objects

StatusSubtypeAssignedTask
OpenNone
OpenNone
OpenNone
OpenNone
DuplicateNone
DeclinedNone
DuplicateJdlrobson
DeclinedNone
ResolvedJdlrobson
Resolved bmansurov
DeclinedNone
ResolvedJdlrobson
InvalidNone
Duplicate Nirzar
OpenNone
ResolvedJdlrobson
DeclinedNone
ResolvedEsanders
ResolvedEsanders
Resolvedmatmarex
Resolvedmatmarex
Resolvedovasileva
ResolvedABorbaWMF
ResolvedNone
DuplicateNone
DuplicateNone
ResolvedJdlrobson
Resolved bmansurov

Event Timeline

bzimport raised the priority of this task from to Needs Triage.Nov 22 2014, 3:13 AM
bzimport set Reference to bz64316.
  • Bug 64029 has been marked as a duplicate of this bug. ***

bingle-admin wrote:

Prioritization and scheduling of this bug is tracked on Trello card https://trello.com/c/OLxnuzXR

At this point I'm opposed to moving this to GeoData because it's a purely data-supplying extension. Instead, a separate extension should be created that would support displaaying POIs not only in a list but also optionally in a map, IMO.

Nemo_bis renamed this task from [Tracking] Move Nearby code to Geodata extension to Move Nearby code to Geodata extension (tracking).Jul 19 2015, 9:59 AM
Jdlrobson renamed this task from Move Nearby code to Geodata extension (tracking) to Move Nearby code to more suitable extension (tracking).Jul 19 2015, 8:33 PM
Jhernandez subscribed.
Jdlrobson renamed this task from Move Nearby code to more suitable extension (tracking) to [EPIC] Move Nearby code to more suitable extension.May 3 2017, 6:03 PM
Jdlrobson lowered the priority of this task from Medium to Low.
Jdlrobson added a project: Epic.
Jdlrobson removed a project: Tracking-Neverending.

@TheDJ what do you think about putting the code into the Maps extension? I see from your email to maps-l you'd like to improve the page and doing this task seems like a good springboard to that....

I think i'm personally in favour of moving it into either Kartographer, or a separate extension..

My biggest concern with putting it into Kartographer, is that I have no feeling for how intertwined that special page, it's JS and it's testcases currently are with MobileFrontend.

My biggest concern with putting it into Kartographer, is that I have no feeling for how intertwined that special page, it's JS and it's testcases currently are with MobileFrontend.

So the main problem is all the JavaScript of Naearby is in MobileFrontend and used by Minerva (and one of the reasons for that dependency)
The only way to avoid this dependency would be to rewrite the special page's JavaScript (maybe using OOjs UI)

If we took this approach we could have a temporary hidden Special page e.g. Special:NearbKartographer, and when it's ready we can simply switch the current special page with the new one.

This might also be a good time to revisit the hacky behaviour of the Nearby refresh icon which replaces the Echo notifications icon - I've always felt that belonged outside the header an how. cc @Nirzar

Am happy to provide responsive code review support for this initiative if you bite!

Change 575568 had a related patch set uploaded (by Jdlrobson; owner: Jdlrobson):
[mediawiki/extensions/MobileFrontend@master] Add compatibility with Extension:NearbyPages

https://gerrit.wikimedia.org/r/575568

Change 575568 merged by jenkins-bot:
[mediawiki/extensions/MobileFrontend@master] Add compatibility with Extension:NearbyPages

https://gerrit.wikimedia.org/r/575568

I've created Extension:NearbyPages
An actionable but stalled task for pursuing this is tracked in T246494