Closed
Bug 1123727
Opened 10 years ago
Closed 10 years ago
[buddyup] Display KB articles with no extra UI for embedding within Buddy Up
Categories
(support.mozilla.org :: Knowledge Base Software, task, P2)
support.mozilla.org
Knowledge Base Software
Tracking
(Not tracked)
RESOLVED
FIXED
2015Q1
People
(Reporter: rehandalal+mozilla, Assigned: rehandalal+mozilla)
References
Details
(Whiteboard: u=user c=BuddyUp p=2 s=2015.2)
Instead of pulling the content of KB articles into Buddy Up we are just going to load the article on the SUMO site and embed within an iframe. This will require getting rid of extra UI (menus etc.) and just showing the HTML of the article.
To keep the number of requests down and generally reduce the size of things, we should only have the HTML, one CSS file for the wiki and the showfor JS.
Updated•10 years ago
|
Priority: -- → P2
Whiteboard: u=user c=BuddyUp p= s=2015.2
Target Milestone: --- → 2015Q1
Comment 1•10 years ago
|
||
This is just copying the current template and deleting things.
Whiteboard: u=user c=BuddyUp p= s=2015.2 → u=user c=BuddyUp p=1 s=2015.2
Assignee | ||
Updated•10 years ago
|
Assignee: nobody → rdalal
Comment 2•10 years ago
|
||
Can we disable the X-Frame-Options: DENY HTTP header for that view?
If we can't, we'll have to ask for the mozbrowser permission for the application to embed this view. I'd like to ask for the smaller amount of permissions possible but I haven't thought about what that implies for us.
Assignee | ||
Comment 3•10 years ago
|
||
Increased to a 2pt'er since the bug has grown in scope and I need to spend some time figuring out how to fix the X-Frame-Options: DENY header for the view.
Whiteboard: u=user c=BuddyUp p=1 s=2015.2 → u=user c=BuddyUp p=2 s=2015.2
Assignee | ||
Comment 4•10 years ago
|
||
Landed:
https://github.com/mozilla/kitsune/commit/e2a8944c5e69bd6925396887865c7bd758b24336
Pushed to prod by me.
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•