Update thimble to publish tutorials as app tags.

RESOLVED WONTFIX

Status

Webmaker
Thimble
RESOLVED WONTFIX
5 years ago
5 years ago

People

(Reporter: thecount, Assigned: thecount)

Tracking

Details

Attachments

(1 attachment)

(Assignee)

Description

5 years ago
There have been changes in the way the makeapi handles and validates tags.

In bug 873605 we updated things to make them work.

In order to get app tags working, I need a way to test this locally and see my tags being published, and I need to make sure I have auth creds to publish app tags.
(Assignee)

Updated

5 years ago
Assignee: nobody → scott

Updated

5 years ago
Status: NEW → ASSIGNED
(Assignee)

Comment 1

5 years ago
Created attachment 762174 [details] [review]
https://github.com/mozilla/thimble.webmaker.org/pull/121

This makes it so only admins can publish tutorials because they are now app tags.

To test it locally go to http://localhost:3000/console (or where ever your login is and give yourslef admins creds)

Brett: I added you just so you know this is happening, can sign off on it, and things may break so now you can know why and get the proper creds if you don't already have them.
Attachment #762174 - Flags: review?(pomax)
Flags: needinfo?(brett)
(Assignee)

Updated

5 years ago
Blocks: 882820
(Assignee)

Comment 2

5 years ago
Comment on attachment 762174 [details] [review]
https://github.com/mozilla/thimble.webmaker.org/pull/121

Looks like we're good for now.

Going to leave this open, and when we decide to lock it out to admins, we'll give anyone the ability to publish tutorials on their own projects if they are not admins.
Attachment #762174 - Flags: review?(pomax)
marking as closed. I like tutorials being open.
Flags: needinfo?(brett)
(Assignee)

Updated

5 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → WONTFIX
Attachment mime type: text/plain → text/x-github-pull-request
You need to log in before you can comment on or make changes to this bug.