Mina Galić igalic
  • Dortmund
  • https://igalic.co/
  • Infrastructure & Open Source

    (Homoiconic Ops Witch)

  • Joined on 2020-06-26
igalic commented on pull request Plume/Plume#805 2020-07-20 12:01:43 +00:00
extract DbConn from PlumeRocket

yesno… we want to retain our ConnectionCustomization, so we're keeping our own init_pool().

igalic commented on pull request Plume/Plume#805 2020-07-20 12:00:01 +00:00
extract DbConn from PlumeRocket
  • pass DbConn in as ref instead of Copying it around.

This will help with the async migration, as we're currently passing all of PlumeRocket around as ref, which means that it ends up sitting somewhere in a function, until it's .awaited.

This can create lead us into a big mess, which may be impossible to untangle, so this should help not running into that to begin with.

igalic pushed to refactor/extract-dbconn at igalic/Plume 2020-07-19 17:02:37 +00:00
3d8785e97b add rocket_contrib as dependency
6de9a1f1c8 Merge pull request 'Recreate search index if its format is outdated' (#802) from KitaitiMakoto/Plume:invalid-index into main
9c7af7dcd4 add rocket_contrib as dependency
5e30bede40 Don't care about needless return value of closures
5cca66b346 Add action user can take to error message
Compare 9 commits »
igalic created pull request Plume/Plume#805 2020-07-19 11:37:21 +00:00
add rocket_contrib as dependency
igalic pushed to refactor/extract-dbconn at igalic/Plume 2020-07-19 11:35:51 +00:00
9c7af7dcd4 add rocket_contrib as dependency
010eac6c4a Merge pull request 'upgrade rocket* to the latest stable' (#800) from igalic/Plume:update/rocket into main
a107d35492 upgrade rocket* to the latest stable
igalic commented on issue Plume/Plume#804 2020-07-19 10:00:41 +00:00
all User facing error messages should be translated

"discovered" in Plume/Plume#802

igalic opened issue Plume/Plume#804 2020-07-19 09:27:42 +00:00
all User facing error messages should be translated
igalic commented on pull request Plume/Plume#802 2020-07-19 08:01:52 +00:00
Recreate search index if its format is outdated

administrators are users too. anyway, let's ear-mark this for a next round

igalic commented on pull request Plume/Plume#802 2020-07-19 06:41:50 +00:00
Recreate search index if its format is outdated

this is a user-facing error message, so it should go thru translation

igalic commented on pull request Plume/Plume#802 2020-07-19 06:41:50 +00:00
Recreate search index if its format is outdated

this is a user-facing error message, so it should go thru translation

igalic commented on pull request Plume/Plume#802 2020-07-19 06:41:50 +00:00
Recreate search index if its format is outdated

is this a user-facing error message? should it go thru translation?

igalic commented on pull request Plume/Plume#802 2020-07-18 19:10:03 +00:00
Recreate search index if its format is outdated

is this clippy still necessary?

igalic commented on pull request Plume/Plume#802 2020-07-18 19:10:03 +00:00
Recreate search index if its format is outdated

any advice on what to do here if this happens?

igalic commented on pull request Plume/Plume#802 2020-07-18 19:10:03 +00:00
Recreate search index if its format is outdated

who is this return for?

igalic commented on pull request Plume/Plume#802 2020-07-18 19:10:03 +00:00
Recreate search index if its format is outdated

maybe we should put the above if let into this match as well…

igalic commented on pull request Plume/Plume#802 2020-07-18 19:10:03 +00:00
Recreate search index if its format is outdated

some comments

igalic commented on issue Plume/Plume#801 2020-07-18 18:49:59 +00:00
[Refactoring] emailing can also be done in an actor!

I've created a basic example with riker and multi-type messages for how to replace Mailer: https://gist.github.com/igalic/119e0f9ab5c3c8f5a3d3f50a5e17ee3a

igalic commented on issue Plume/Plume#799 2020-07-18 18:49:12 +00:00
[Refactoring] send messages to dedicated actor instead of directly accessing objects

I've created a basic example with riker and multi-type messages for how to replace Searcher: https://gist.github.com/igalic/16ec36a4c9277b3b46f6bea032226189

in a similar fashion, we can replace Worker, which currently mostly sents broadcast messages to all subscribers.

igalic commented on pull request Plume/Plume#802 2020-07-18 10:36:56 +00:00
Recreate search index if its format is outdated

why would we need to keep our restore an index which Plume cannot load?

what could the admin do, that we can't? (why aren't we doing that?)