Fix follow IDs #455

Scalone
elegaanz scala 4 commity/ów z fix-follow-id do master 5 lat temu
elegaanz skomentował(-a) 5 lat temu (Zmigrowane z github.com)

I changed the syntax of the insert! macro, allowing to modify an object just after it was saved to the database. You can now optional pass it a "closure" (actually it just looks like a closure) that will be run just after the insertion.

I updated all the models, including Follow, to use it to generate AP ID and URL. The advantages are that the ID/URL are now correct (fixes #449) and that we don't have methods like update_boxes or update_ap_url to call manually and that were easy to forget.

I changed the syntax of the `insert!` macro, allowing to modify an object just after it was saved to the database. You can now optional pass it a "closure" (actually it just looks like a closure) that will be run just after the insertion. I updated all the models, including Follow, to use it to generate AP ID and URL. The advantages are that the ID/URL are now correct (fixes #449) and that we don't have methods like `update_boxes` or `update_ap_url` to call manually and that were easy to forget.
elegaanz (Zmigrowane z github.com) zrecenzowano 5 lat temu
@ -228,4 +228,3 @@
post.cover_id = form.cover;
post.update(&*conn, &searcher).expect("post::update: update error");;
let post = post.update_ap_url(&*conn).expect("post::update: update ap url error");
elegaanz (Zmigrowane z github.com) skomentował(-a) 5 lat temu

Just a note: I removed that without replacing it with something else, since it is incorrect (the AP IDs should change), and was not working anyway (since we were checking if ap_url was empty before updating it, which was only the case on newly created articles).

Just a note: I removed that without replacing it with something else, since it is incorrect (the AP IDs should change), and was not working anyway (since we were checking if `ap_url` was empty before updating it, which was only the case on newly created articles).
codecov[bot] skomentował(-a) 5 lat temu (Zmigrowane z github.com)

Codecov Report

Merging #455 into master will increase coverage by 1.41%.
The diff coverage is 79.31%.

@@            Coverage Diff             @@
##           master     #455      +/-   ##
==========================================
+ Coverage   25.06%   26.48%   +1.41%     
==========================================
  Files          64       64              
  Lines        6316     7111     +795     
==========================================
+ Hits         1583     1883     +300     
- Misses       4733     5228     +495
# [Codecov](https://codecov.io/gh/Plume-org/Plume/pull/455?src=pr&el=h1) Report > Merging [#455](https://codecov.io/gh/Plume-org/Plume/pull/455?src=pr&el=desc) into [master](https://codecov.io/gh/Plume-org/Plume/commit/a2b9d7ec44daf91d5b0dde00715d7185b5342c94?src=pr&el=desc) will **increase** coverage by `1.41%`. > The diff coverage is `79.31%`. ```diff @@ Coverage Diff @@ ## master #455 +/- ## ========================================== + Coverage 25.06% 26.48% +1.41% ========================================== Files 64 64 Lines 6316 7111 +795 ========================================== + Hits 1583 1883 +300 - Misses 4733 5228 +495 ```
trinity-1686a zatwierdza te zmiany 5 lat temu
trinity-1686a zostawił komentarz
Właściciel

LGTM

LGTM
trinity-1686a skomentował(-a) 5 lat temu
Właściciel

I've never used this syntax, but it looks super nice

I've never used this syntax, but it looks super nice
trinity-1686a skomentował(-a) 5 lat temu
Właściciel

this route should really get implemented one day

this route should really get implemented one day
elegaanz (Zmigrowane z github.com) zrecenzowano 5 lat temu
elegaanz (Zmigrowane z github.com) skomentował(-a) 5 lat temu

Yes, I think we already have an issue for it…

Yes, I think we already have an issue for it…
igalic (Zmigrowane z github.com) zrecenzowano 5 lat temu
igalic (Zmigrowane z github.com) skomentował(-a) 5 lat temu

what does it do?

what does it do?
elegaanz (Zmigrowane z github.com) zrecenzowano 5 lat temu
elegaanz (Zmigrowane z github.com) skomentował(-a) 5 lat temu

For all the fields that were not provided, it takes their value from the given struct (here NewBlog::default()).

For all the fields that were not provided, it takes their value from the given struct (here `NewBlog::default()`).

Recenzenci

trinity-1686a zatwierdza te zmiany 5 lat temu
Pull Request został scalony jako 2a188abfa1.
Możesz także zobaczyć instrukcje wiersza poleceń.

Krok 1:

Z repozytorium twojego projektu, sprawdź nową gałąź i przetestuj zmiany.
git checkout -b fix-follow-id master
git pull origin fix-follow-id

Krok 2:

Połącz zmiany i zaktualizuj na Forgejo.
git checkout master
git merge --no-ff fix-follow-id
git push origin master
Zaloguj się, aby dołączyć do tej rozmowy.
Brak recenzentów
Brak kamienia milowego
Brak przypisanych
Uczestnicy 2
Powiadomienia
Termin realizacji
Data realizacji jest niewłaściwa lub spoza zakresu. Użyj formatu 'yyyy-mm-dd'.

Brak ustawionego terminu realizacji.

Zależności

No dependencies set.

Reference: Plume/Plume#455
Ładowanie…
Nie ma jeszcze treści.