Refactor code and remove akita-sidebar comments in sidebar.jsx

RESOLVED FIXED

Status

P1
normal
Rank:
11
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: mancas, Assigned: mancas)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [akita])

User Story

* Tech list:
* Refactor sidebar.jsx code
** initialization code
** remove useless functions/stores/whatever
* Re-write or re-use conversation tests

Attachments

(1 attachment)

(Assignee)

Description

3 years ago
Clean up sidebar code especially the initialization part. The useless code should be removed and the "test" stuff fixed. Also the conversation tests needs to be rewritten to fit the new sidebar approach.
Created attachment 8748514 [details] [review]
[loop] mancas:bug1268811 > mozilla:akita
(Assignee)

Updated

3 years ago
Attachment #8748514 - Flags: review?(standard8)
Comment on attachment 8748514 [details] [review]
[loop] mancas:bug1268811 > mozilla:akita

I've responded to the questions on the PR. I think there's nothing else here that needs my specific review, so I'll punt on that, and let you request review from other people when you're ready.
Attachment #8748514 - Flags: review?(standard8)
Rank: 11
(Assignee)

Comment 3

3 years ago
Comment on attachment 8748514 [details] [review]
[loop] mancas:bug1268811 > mozilla:akita

I was getting an error when running |make test| but it seems to be a timing problem so if you face to it, raise your hand

The error:
http://pastebin.com/v34nzQDX
Attachment #8748514 - Flags: review?(dmose)
Attachment #8748514 - Flags: review?(crafuse)
Comment on attachment 8748514 [details] [review]
[loop] mancas:bug1268811 > mozilla:akita

Stealing reviews, as looking at other bugs, I wanted some of this to be there anyway, so I thought I'd just review it.

Looks good, r=Standard8
Attachment #8748514 - Flags: review?(dmose)
Attachment #8748514 - Flags: review?(crafuse)
Attachment #8748514 - Flags: review+
(Assignee)

Comment 5

3 years ago
Landed in akita branch: https://github.com/mozilla/loop/commit/123968f9ebbb087e0b2f02d5b7ae4e847b7bfdff
Status: ASSIGNED → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.