Plume
/
Plume
8
16
Форк 22
Код Задачі 174 Запити на злиття 8 Релізи 8 Вікі Активність

Мітки Етап

extract DbConn from PlumeRocket #805

Закрито
igalic хоче злити 2 комітів з igalic/Plume:refactor/extract-dbconn в main
pull з: igalic/Plume:refactor/extract-dbconn
злити в: Plume:main
Обговорення 2 Коміти 2 Змінені файли 6
igalic прокоментував(ла) 4 роки тому
Власник

in the following Pull Request, we'lle extract DbConn from PlumeRocket, as suggested in #797

  • use rocket_contrib's #[database] to implement our DbConn
  • we do not use #[database]'s connection setup, retaining our own init_pool() and ConnectionCustomization!
  • pass DbConn in as ref instead of Copying it around.
in the following Pull Request, we'lle extract `DbConn` from `PlumeRocket`, as suggested in https://git.joinplu.me/Plume/Plume/issues/797 - use rocket_contrib's `#[database]` to implement our `DbConn` - we do *not* use `#[database]`'s connection setup, retaining our own `init_pool()` and `ConnectionCustomization`! - pass `DbConn` in as `ref` instead of `Copy`ing it around.
👍 1
kiwii рецензовано 4 роки тому
plume-models/src/db_conn.rs Застарілі
@ -41,3 +12,1 @@
&self.0
}
}
#[database("plume")]
kiwii прокоментував(ла) 4 роки тому
Власник

I think you'll have to edit plume-models/src/config.rs so that Rocket knows the URL of this "plume" database.

I think you'll have to edit `plume-models/src/config.rs` so that Rocket knows the URL of this "plume" database.
igalic прокоментував(ла) 4 роки тому
Автор
Власник

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

yesno… we want to retain our `ConnectionCustomization`, so we're keeping our own `init_pool()`.
igalic позначив обговорення завершеним
igalic примусово залито refactor/extract-dbconn з %!<(string=9c7af7dcd4)/code> до 3d8785e97b 4 роки тому
igalic змінився заголовок з add rocket_contrib as dependency на extract DbConn from PlumeRocket 4 роки тому
igalic прокоментував(ла) 4 роки тому
Автор
Власник
  • 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.

> - pass `DbConn` in as `ref` instead of `Copy`ing 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 `.await`ed. 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 додав 1 коміт 4 роки тому
e19069f750
remove DbConn from PlumeRocket 
this means compilation fails, and for the next while, we'll try to make
it compile again ;)
igalic прокоментував(ла) 4 роки тому
Автор
Власник

closing in favour of #813

closing in favour of #813
igalic закрив цей запит на злиття 4 роки тому
Будь ласка перевідкрийте цей запит щоб здіснити операцію злиття.
Підпишіться щоб приєднатися до обговорення.
Рецензенти
Попросити рецензію
Немає рецензентів
Мітки
Застосувати мітки
Очистити мітки   
A: API

Related to the REST API   
A: Backend

Code running on the server   
A: Federation

Stuff related to Federation   
A: Front-End

Related to the front-end   
A: I18N

Translations, and related code   
A: Meta

More about project management or code than the project itself   
A: Security
   
Build

The building, or installation process of Plume   
C: Bug

Something isn't working   
C: Discussion

We need to talk   
C: Enhancement

New feature or request   
C: Feature

This is a new feature   
Compatibility

Compatibility with different browsers, readers and OS   
Dependency

Related to an external package that Plume uses   
Design

UI/UX related issues and PRs   
Documentation
   
Good first issue

Good for newcomers   
Help welcome

Extra attention is needed   
Mobile

Issues affecting only mobile UX   
Rendering

How elements're rendered out for the end user   
S: Blocked

Something else needs to be fixed first   
S: Duplicate

This issue or pull request already exists   
S: Incomplete

This PR is not complete yet   
S: Instance specific

Issues concern a limited number of instances   
S: Invalid

This doesn't seem right   
S: Needs Voting/Discussion

Need to be discussed by the community (on Loomio)   
S: Ready for review

This PR is ready to be reviewed   
Suggestion

Proposed ideas worth considering   
S: Voted on Loomio

This is issue has been created after a vote on Loomio   
S: Wontfix

This will not be worked on
Без мітки
A: API
A: Backend
A: Federation
A: Front-End
A: I18N
A: Meta
A: Security
Build
C: Bug
C: Discussion
C: Enhancement
C: Feature
Compatibility
Dependency
Design
Documentation
Good first issue
Help welcome
Mobile
Rendering
S: Blocked
S: Duplicate
S: Incomplete
S: Instance specific
S: Invalid
S: Needs Voting/Discussion
S: Ready for review
Suggestion
S: Voted on Loomio
S: Wontfix
Етап
Призначити етап
Очистити етап
Немає елементів
Етап відсутній
Виконавці
Призначити користувачів
Прибрати виконавців
Немає виконавця
2 учасників
Сповіщення
Дата завершення
Термін дії не дійсний або знаходиться за межами допустимого діапазону. Будь ласка використовуйте формат 'yyyy-mm-dd'.

Термін виконання не встановлений.

Залежності

No dependencies set.

Reference: Plume/Plume#805
Писати Попередній перегляд
Завантаження…
Відмінити
Зберегти
Тут ще немає жодного змісту.
Видалити гілку 'igalic/Plume:refactor/extract-dbconn'

Видалення гілки НЕЗВОРОТНЕ. Дію не можна скасувати. Продовжити?

Ні
Так