Closed
Bug 758730
Opened 13 years ago
Closed 4 years ago
Remove MindTouch dependencies entirely
Categories
(developer.mozilla.org Graveyard :: Code Cleanup, defect)
developer.mozilla.org Graveyard
Code Cleanup
Tracking
(Not tracked)
RESOLVED
WONTFIX
People
(Reporter: lorchard, Unassigned)
Details
Once bug 757940 is closed, we should be able to turn off anything that integrates with MindTouch.
But, the code will still be around. So, once we've shut down the last MindTouch server for good, we should comb through and remove all integration points entirely.
Reporter | ||
Comment 1•13 years ago
|
||
* apps/dekicompat - Remove the entire module, see what breaks
* apps/devmo/models.py - Remove UserDocsActivityFeed code, superseded by Kuma models
* apps/devmo/helpers.py - Remove devmo_url() helper, see what breaks
* apps/docs/cron.py - Remove feed code, see bug 759361 and bug 759368
Reporter | ||
Comment 2•13 years ago
|
||
* apps/devmo/models.py - UserProfile.save() wants to POST to the MindTouch API
Assignee | ||
Updated•12 years ago
|
Version: Kuma → unspecified
Assignee | ||
Updated•12 years ago
|
Component: Docs Platform → Editing
Updated•11 years ago
|
Component: Editing → General
Comment 3•11 years ago
|
||
I'm seeing this in the save method:
if 'skip_mindtouch_put' in kwargs:
del kwargs['skip_mindtouch_put']
Do we know if we can quickly whack this?
Flags: needinfo?(lorchard)
Reporter | ||
Comment 4•11 years ago
|
||
(In reply to David Walsh :davidwalsh from comment #3)
> I'm seeing this in the save method:
>
> if 'skip_mindtouch_put' in kwargs:
> del kwargs['skip_mindtouch_put']
>
> Do we know if we can quickly whack this?
Hmm, I can't find what you're referring to... Got a link to it in github?
Flags: needinfo?(lorchard)
Comment 5•11 years ago
|
||
Reporter | ||
Comment 6•11 years ago
|
||
Yeah, I think this can get taken out. Probably fine, assuming tests all pass. This might be a leftover from the migration way back when
Comment 7•10 years ago
|
||
The line referenced in comment 5 is no longer there. It doesn't appear to be anywhere[0].
Is this bug still outstanding?
[0] https://github.com/mozilla/kuma/search?q=skip_mindtouch_put&type=Code&utf8=%E2%9C%93
Comment 8•7 years ago
|
||
There are some remaining MindTouch pieces:
https://github.com/mozilla/kuma/search?utf8=%E2%9C%93&q=mindtouch&type=
Some remaining actions:
* Determine if the MindTouch table sort CKEditor plugin [1] still in use, remove if unused
* Update pages with MindTouch-style attachment URLs to new attachment URLs. For example, the :hover page [2] has an old link [3] that redirects to the new link [4].
* Remove the attachment view [5] once old-style attachment URLs have been converted for at least 30 days, to let Google catch up.
* Remove mindtouch IDs from the database [6]
* Update legacy MindTouch links to new link locations. For example, the page User:Diablownik has a link to a tag page [8] that redirects to the new page [9] (this is a poor example, but is the first of many I found).
* Remove the MindTouch redirect view [10] once old-style URLs have been converted for at least 30 days.
* Evaluate other MindTouch instances in the content.
All of this is doable. It could be a manual effort (generate spreadsheet with actions), and some could be done with automation.
[1] https://github.com/mozilla/kuma/blob/00fc05b101658f863f58d7fc2b1aefecbcf71cf8/kuma/static/js/libs/ckeditor/source/plugins/tablesort/plugin.js
[2] https://developer.mozilla.org/en-US/docs/Web/CSS/:hover
[3] https://developer.mozilla.org/@api/deki/files/6247/=css-gallery.zip
[4] https://mdn.mozillademos.org/files/3707/css-gallery.zip
[5] https://github.com/mozilla/kuma/blob/61aea62e532e391c5c8e26e0eaa5049d194ee0ad/kuma/attachments/views.py#L60-L64
[6] https://github.com/mozilla/kuma/blob/79e24a24920e05344096e049c586ad8aecf58038/kuma/attachments/models.py#L129-L139
[7] https://developer.mozilla.org/en-US/docs/User:Diablownik
[8] https://developer.mozilla.org/Special:Tags?tag=Strony_wymagaj%C4%85ce_dopracowania&language=pl
[9] https://developer.mozilla.org/en-US/docs/tags
[10] https://github.com/mozilla/kuma/blob/286948a9ba70454914eacf983015197500c05419/kuma/wiki/views/legacy.py#L24-L56
Component: General → Code Cleanup
Comment 9•4 years ago
|
||
MDN Web Docs' bug reporting has now moved to GitHub. From now on, please file content bugs at https://github.com/mdn/sprints/issues/ and platform bugs at https://github.com/mdn/kuma/issues/.
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → WONTFIX
Updated•4 years ago
|
Product: developer.mozilla.org → developer.mozilla.org Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•