Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Activity
  • Sep 14 18:44
    Travis jasonmunro/cypht (master) passed (1063)
  • Sep 14 18:33
    Travis jasonmunro/cypht (master) passed (1058)
  • Sep 14 18:29
    Travis jasonmunro/cypht (master) canceled (1062)
  • Sep 14 18:29

    jasonmunro on master

    Fixed combined inbox not updati… Merge pull request #507 from he… (compare)

  • Sep 14 18:28
    Travis jasonmunro/cypht (master) canceled (1061)
  • Sep 14 18:28

    jasonmunro on master

    Check the folder name without t… Merge pull request #509 from Wi… (compare)

  • Sep 14 18:27
    Travis jasonmunro/cypht (master) canceled (1060)
  • Sep 14 18:27

    jasonmunro on master

    fix of the overflow of array el… Merge pull request #510 from Na… (compare)

  • Sep 14 18:23
    Travis jasonmunro/cypht (master) canceled (1059)
  • Sep 14 18:23

    jasonmunro on master

    Draft Folder option, select fol… Merge pull request #513 from Na… (compare)

  • Sep 14 18:22

    jasonmunro on master

    Settings, Folder management, co… Merge pull request #514 from Na… (compare)

  • Sep 14 18:21

    jasonmunro on master

    Typo fix, added in call to apt … Added quick test to see if comp… Merge pull request #517 from je… (compare)

  • Sep 03 16:17
    Travis jasonmunro/cypht (master) passed (1057)
  • Sep 03 16:10

    jasonmunro on master

    fix profile re-include causing … (compare)

  • Sep 01 21:08
    Travis jasonmunro/cypht (master) passed (1056)
  • Sep 01 21:01

    marclaporte on master

    We now have a stable branch on … (compare)

  • Aug 24 18:27
    Travis jasonmunro/cypht (master) passed (1055)
  • Aug 24 18:20

    jasonmunro on master

    Profiles counter on home page Removed duplicated SMTP profile… Added $profiles default value and 1 more (compare)

  • Aug 17 18:59
    Travis jasonmunro/cypht (master) passed (1054)
  • Aug 17 18:52

    jasonmunro on master

    Ignoring .idea folder Merge branch 'master' of github… Filter folder page to show sent… and 1 more (compare)

Weston Miller
@kk6mrp
Awesome, thanks Jason!
Marc Laporte
@marclaporte_gitlab
For the record: @Revisto and I discussed about making a demo space for Cypht
3 actually
1- Demo main branch (master)
2- Demo stable branch (1.3.x)
3- Demo the new "Revisto" theme
We will keep you all posted.
Marc Laporte
@marclaporte_gitlab
We will put a server limitation on the number of emails sent, so spammers can't abuse it.
Jason Munro
@jasonmunro
I would limit outbound E-mail to appear to work but not actually send like with a dummy SMTP server
Marc Laporte
@marclaporte_gitlab
Right, we can do that but it should be clearly indicated to the user.
So @jasonmunro and @henrique-borba : How much old draft code remains in the code? And if we remove it, can it cause issues?
Ref: jasonmunro/cypht#508
Jason Munro
@jasonmunro
not a lot I don't think. All changes can cause issues of course, but removing it is the right thing to do and will result in less issues once it's done I think
Marc Laporte
@marclaporte_gitlab
ok, thanks, so sounds like best is for @henrique-borba to rip out all old draft code and then review the new draft code he made with a special focus on attachments.
Jason Munro
@jasonmunro
yep
Henrique Borba
@henrique-borba
I removed the old code, I'm testing it now and working on the draft ID's. I guess #508 won't need to be merged after all
Marc Laporte
@marclaporte_gitlab
Thank you @henrique-borba !
Henrique Borba
@henrique-borba
Hey guys, after my work hours I will open the PR's =D
Marc Laporte
@marclaporte_gitlab
Super!
Here are 3 PRs with cosmetic enhancements, with before and after screenshots:
jasonmunro/cypht#514
jasonmunro/cypht#513
jasonmunro/cypht#510
Jason Munro
@jasonmunro
I know the PRs are stacking up, appreciate all the work everyone. I will make some time this week to review/approve!
Weston Miller
@kk6mrp
Is there pending issues with attachment support? I can attach a file but it doesn't send with the email. Is there a certain amount of time you have to wait before you send because it almost seems like the drafts support is picking up a slightly older version and sending that one... I can't prove that though.
Marc Laporte
@marclaporte_gitlab
9 PRs from 6 developers: I think this is an all time record :-)
@kk6mrp: @henrique-borba has a PR on the way to remove all the old attachment code, and make sure all works with the new
Part of the solution is here: jasonmunro/cypht#508
Jason Munro
@jasonmunro
it got to 10 actually :) down to 4 awaiting updates or feedback
Marc Laporte
@marclaporte_gitlab
Well done Jason and all the committers!
FYI, the integration of Cypht with the Tiki database component (Trackers) is progressing well.
In the latest version (23.0) we can now move emails back and forth between IMAP and database storage.
We found 6-7 minor integration bugs which we'll resolve in coming weeks, after which, we'll use in production with a team of 10 people.
Marc Laporte
@marclaporte_gitlab
This more intense usage will lead to more bug reports and feature requests.
Jason Munro
@jasonmunro
nice!
Marc Laporte
@marclaporte_gitlab
Team has access to 4 IMAP mailboxes and they move mails to relevant task (where it gets stored in DB)
For now, we move everything manually, but the pressure will progressively increase for server-side filtering.
Specifically, when email is in a task, anyone on that task can reply to the mail. We need the reply back to make its way back to the inbox of the task.
This forces us to tackle the oldest still open feature request: jasonmunro/cypht#9
Marc Laporte
@marclaporte_gitlab
We could do all this in Tiki and it would actually be easier, but I'd much prefer doing in Cypht and we call it from Tiki.
Realistically, this will be top of list in 2-3 months, so please everyone add your ideas to that issue.
Marc Laporte
@marclaporte_gitlab
@jasonmunro If you like my idea at jasonmunro/cypht#516 I'll ask a junior dev to code it.
Jason Munro
@jasonmunro
@marclaporte_gitlab sure that works, thanks!
Marc Laporte
@marclaporte_gitlab
ok, I added it for adoption for the young guns :-)
Marc Laporte
@marclaporte_gitlab
Thank you @neildarlow for sharing how this is already an annoyance for you jasonmunro/cypht#516 and thus, it reinforces proposed course of action.
@henrique-borba informed me in private that he is working hard on the draft/attachment issue and there are some edge cases.
~jonah | Mineman
@jonah_mineman_twitter
Hello there, does a hub or something for themes exist?
Marc Laporte
@marclaporte_gitlab
You for themes that are not bundled in Cypht?
~jonah | Mineman
@jonah_mineman_twitter
Yes
Jason Munro
@jasonmunro
there is nothing like that at the moment but it is a cool idea. I would be able to host a site for that easily enough - though I'm also happy to include any contributed theme in the main source as well. It would be cool to have screenshots of each and more themes are always appreciated
~jonah | Mineman
@jonah_mineman_twitter
Cool
Henrique Borba
@henrique-borba
Hey @jasonmunro, Marc updated you. Just to be more precise. I got stuck into a attachment problem were at first I tought it was a bug but it was just the filesystem cache that was getting cleaned. I decided to "recheck" for draft attachments everytime the compose page opens a draft email, it compares with the current cache and "reupload" (download and upload) the attachments that are not in the cache in the newer draft version. (Drafts are just a seek-and-delete overwrite system, so the attachments for the newer draft version must always be reuploaded)
marclaporte:matrix.org @marclaporte:matrix.org testing this room via Matrix....