892033 - Make Editor - add direct link to Popcorn remix

Status: RESOLVED FIXED

(Webmaker Graveyard :: webmaker.org, defect)

Description

[Jacob [:Jacob]]

Attachment: Screenshot explaining proposed change

Hey Chris - I'm realizing that when I'm skimming through projects to see if they're good for a feature, I have better odds of knowing what's good if I'm watching it in the editor mode.

The reason is because from the editor mode, I can tell right away from a glance how many events the person is using, and thus better forecast if it's a good project to feature.

So - where would be a good place to link to a project's /remix URL from the Make Editor?

I'd much prefer if we can avoid having to make a new column for this...is there any way to link to it from one of the existing columns?

Ideally - we could change the first URL column so that instead of pointing to the export URL, i.e. https://sdavies.makes.org/popcorn/18ec like it does currently...

We instead point it to the actual remix URL, i.e. https://popcorn.webmaker.org/editor/57540/remix

THEN we could make the title of the project be a live hyperlink to the export page
But just keep the anchor text as the Title

(see attached screenshot if that doesn't make sense)

Comment 1

[Chris DeCairos (:cade)]

This is easy to do, and sounds like a good idea.

The one problem I forsee is that it won't scale to third party makes, because we can't assume they'll have a /remix feature implemented.

That said, what do you think?

Comment 2

[Jacob [:Jacob]]

So if I understand - by "third party makes" you're referring to Makes that don't come from our tools but are hooked in to the MakeAPI and are being published to webmaker.org?

I guess if they don't have a /remix URL then it can be left blank or just a message that that states it's a third party, or better yet, states who the third party is.

Would that work?

Comment 3

[Chris DeCairos (:cade)]

I could do some sort of URL parsing to determine if it's hosted on makes.org

My focus right now is on the push to third party makes, so this isn't going to get prioritized unless you can convince me its mission critical :)

Comment 4

[Chris DeCairos (:cade)]

Attachment: https://github.com/mozilla/MakeAPI/pull/137

Comment 5

[Matthew Schranz [:mjschranz]]

Comment on attachment 796039 [details] [review]
https://github.com/mozilla/MakeAPI/pull/137

Space nit.

Also, this seems to be missing the requested feature for the title.

Comment 6

[Chris DeCairos (:cade)]

Comment on attachment 796039 [details] [review]
https://github.com/mozilla/MakeAPI/pull/137

fixed!

Comment 7

[Chris DeCairos (:cade)]

Comment on attachment 796039 [details] [review]
https://github.com/mozilla/MakeAPI/pull/137

Why didn't I merge this? I cannot say why... but I've re-based it and would like you to re-review.

Comment 8

[Matthew Schranz [:mjschranz]]

Comment on attachment 796039 [details] [review]
https://github.com/mozilla/MakeAPI/pull/137

R+

Comment 9

[[github robot]]

Commit pushed to master at https://github.com/mozilla/MakeAPI

https://github.com/mozilla/MakeAPI/commit/04c9968b031f2b48b21729d98cf9d228b160ddde
Fix bug 892033 - Change formatting function for URL field so that it links to a remix of the project