Page MenuHomePhabricator

MetaMTA - more progress towards a mail application
ClosedPublic

Authored by btrahan on Jun 22 2015, 11:49 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Mar 23, 2:35 PM
Unknown Object (File)
Sat, Mar 23, 1:01 PM
Unknown Object (File)
Thu, Mar 21, 7:05 AM
Unknown Object (File)
Wed, Mar 20, 4:57 PM
Unknown Object (File)
Wed, Mar 20, 2:35 PM
Unknown Object (File)
Feb 12 2024, 2:37 AM
Unknown Object (File)
Feb 10 2024, 4:13 AM
Unknown Object (File)
Feb 3 2024, 5:17 AM
Subscribers

Details

Summary

Ref T5791. This diff does a few things...

  • Adds code to write recipients to edges on save
  • Makes Query performance for policy filtering okay-ish
  • Adds a Search Engine for PhabricatorMetaMTAMail
  • Adds "working" List Controller
    • Inbox and Outbox both work
  • Adds stub View Controller
Test Plan

ran ./bin/storage upgrade and saw my inbox and outbox start getting data. played with application and saw new entries in inbox and outbox

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

btrahan retitled this revision from to MetaMTA - more progress towards a mail application.
btrahan updated this object.
btrahan edited the test plan for this revision. (Show Details)
btrahan added a reviewer: epriestley.

I just made the recipient edge one way; I figured thats all we needed.

epriestley edited edge metadata.
epriestley added inline comments.
resources/sql/autopatches/20150622.metamta.7.recipient-edge-mig.php
1 ↗(On Diff #32438)

I'd maybe just drop this since I'm not sure it's worth the downtime.

src/applications/metamta/storage/PhabricatorMetaMTAMail.php
375

Possibly we should write edges to the union of the expanded and un-expanded recipients, so you can search for stuff sent to projects. We could do that later.

1043

This potentially involves queries which will do single gets, so we should maybe load + attach the edges in the Query unconditionally to make this cheap. Not too important for now.

This revision is now accepted and ready to land.Jun 23 2015, 12:36 AM

I just made the recipient edge one way; I figured thats all we needed.

Yeah, I don't see a use for the other half of the edge either.

ACK'ing the comments; thanks!

resources/sql/autopatches/20150622.metamta.7.recipient-edge-mig.php
1 ↗(On Diff #32438)

Alrighty, i'll rm the script. Was still good ish for development. :D

src/applications/metamta/storage/PhabricatorMetaMTAMail.php
375

will address in this diff.

1043

I'll look at this in a later diff.

btrahan edited edge metadata.
  • remove migration script
  • add project phids, etc to edges
This revision was automatically updated to reflect the committed changes.