Page MenuHomePhabricator

Move CopyPatrol tool to new URL
Closed, ResolvedPublic2 Estimated Story Points

Description

Let's set up the CopyPatrol tool at http://tools.wmflabs.org/copypatrol (which is easier to remember and spell) and have a redirect from https://tools.wmflabs.org/plagiabot/. This may require redoing some of the OAuth set-up.

Event Timeline

kaldari triaged this task as Medium priority.Jun 7 2016, 4:51 PM
kaldari moved this task from New & TBD Tickets to Needs Discussion on the Community-Tech board.
kaldari added a project: CopyPatrol.
DannyH set the point value for this task to 2.Jun 7 2016, 5:49 PM
DannyH edited projects, added Community-Tech-Sprint; removed Community-Tech.

Steps I followed here:

  • Register new tool
  • SSH to tool
  • Set remote as https://github.com/Niharika29/PlagiabotWeb.git and do a git pull
  • Run 'composer install'
  • Filed a new OAuth consumer request and added credentials and other variables to .env file
  • Do a 'webservice start'

Looks good. Can you replace https://tools.wmflabs.org/plagiabot/ with a message saying the tool has been moved to https://tools.wmflabs.org/copypatrol/?

Looks good. Can you replace https://tools.wmflabs.org/plagiabot/ with a message saying the tool has been moved to https://tools.wmflabs.org/copypatrol/?

Actually @MusikAnimal and I thought it'd be a good idea to use this instance as a testing ground for the actual tool. I'm trying to come up with a way to have a banner on top saying that this is a test instance and the real tool is at the other URL. It's tricky because the code is the same for both.

@Niharika: You could just use...

// Show warning if we are on test site.
if ( window.location.href.indexOf( "tools.wmflabs.org/plagiabot" ) !== -1 ) {
  // banner code
}

... to make it simple. It might be a little hacky having a URL there rather than abstracting it, but I don't think it's worth having more than a few lines of code for this.

I actually added this in my most recent PR here (I think this will work) and here. This also hides the "This is a web-version of Plagiabot" message, since that would seem to imply they're in the right place. JavaScript works just as well, though :)

I adopted an easier (lazier?) solution as you can see on https://tools.wmflabs.org/copypatrol/
@kaldari you think that suffices?

@Niharika: I don't know. I think people are going to be confused, especially since both sites are titled "CopyPatrol" in the header. Also, it's kind of a waste to make everyone that comes to /copypatrol/ read about the difference in the two sites, since they are unlikely to care about or visit /plagiabot/. What if we just had an if statement that changed the text in the header from "CopyPatrol" to "CopyPatrol (experimental version)" if you're on /plagiabot/, that should be pretty easy to implement.

@kaldari Leon did this per his earlier comment. Can be seen on our test instance.

@Niharika: The notice at /plagiabot/ looks good. Can we remove the notice at /copypatrol/ though? Eventually we're going to have to replace "This is a web-version of Plagiabot." with an actual explanation of the tool, and I'm worried about that banner getting too long and no one actually reading it. I think just having a notice at /plagiabot/ is sufficient.