Open
Bug 561617
Opened 15 years ago
Updated 2 years ago
Add Semester and Quaterly Archive possibility
Categories
(Thunderbird :: Mail Window Front End, enhancement)
Tracking
(Not tracked)
ASSIGNED
People
(Reporter: chrissc.humbert, Assigned: chrissc.humbert)
References
(Blocks 1 open bug)
Details
Attachments
(1 file, 1 obsolete file)
(deleted),
patch
|
Bienvenu
:
review-
|
Details | Diff | Splinter Review |
User-Agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9.2.3) Gecko/20100401 Firefox/3.6.3
Build Identifier:
Hello,
Following bug 522761, I would like to add the possibility to manage Archive at semester and quarter level.
Patch is included. Mozmill will follow.
Reproducible: Always
Assignee | ||
Comment 1•15 years ago
|
||
Hello,
Here is the patch to take care of archive level at semester level and quarter level.
Tested. Does not break previous way of working.
Patch is taking also care of the namespace problem with Inbox on bug 522761
I will add mozmill tests later on.
Updated•15 years ago
|
Comment 2•15 years ago
|
||
(In reply to comment #1)
> Created an attachment (id=441347) [details]
> Semester and Quarter Archive
Thanks for the patch, as you can see at https://developer.mozilla.org/en/comm-central#Requirements you'll need to set a review request flag to get your patch reviewed. If you do change the UI you'll also need to set a UI review request.
Assignee | ||
Comment 3•15 years ago
|
||
Hello Ludovic,
I know thta but as mozmill tests are not yet included I prefer to wait a bit before requesting review.
What do you think?
Kind Regards
Comment 4•15 years ago
|
||
(In reply to comment #3)
> Hello Ludovic,
>
> I know thta but as mozmill tests are not yet included I prefer to wait a bit
> before requesting review.
> What do you think?
Wait for complete patch + tests yes. I wasn't sure so I gave you the docs ;-)
Assignee | ||
Comment 5•15 years ago
|
||
Hello,
Here is the full patch. I don't know why I have still some changes for files I did not touch for this bug.
I haev an issue with my hg but I have done hg pull and hg update and hg commit before creating the diff but still no changes.
Nevertheless this should applies correctly.
Kind Regards
Attachment #441347 -
Attachment is obsolete: true
Attachment #441506 -
Flags: review?(bienvenu)
Comment 6•14 years ago
|
||
Comment on attachment 441506 [details] [diff] [review]
Full Patch with semiannually and qauterly archive with mozmill tests
I think this would be better done as an extension.
Comment 7•14 years ago
|
||
Comment on attachment 441506 [details] [diff] [review]
Full Patch with semiannually and qauterly archive with mozmill tests
minusing based on that...
Attachment #441506 -
Flags: review?(bienvenu) → review-
Assignee | ||
Comment 8•14 years ago
|
||
Hello David,
I had another idea.
Replace the granularity integer by a string of 4 characters allowing to have something like
1.YSQM saying ok I want my archives to be organized with (Y)yearly,(S) semi annually, (Q) quaterly, (M) monthly folders
2.YM saying ok I want my archives to be organized with yearly, monthly folders
etc...
Do you think it is a valid idea. I think it will bring great flexibility on the archive feature of TB
Kind Regards
Comment 9•14 years ago
|
||
Christophe, if you do this as an extension, you can try out whatever ideas you like. You could experiment with using nsIMsgIncomingServer's get/set CharValue method to set a string property describing the archive structure, and your extension could read that string property and implement the archive granularity described by the string.
Updated•2 years ago
|
Severity: normal → S3
You need to log in
before you can comment on or make changes to this bug.
Description
•