Post creation API #307

Samengevoegd
elegaanz heeft 9 commits samengevoegd van post-api naar master 5 jaren geleden
elegaanz reageerde 5 jaren geleden (Gemigreerd van github.com)
Er is nog geen inhoud.
codecov[bot] reageerde 5 jaren geleden (Gemigreerd van github.com)

Codecov Report

Merging #307 into master will decrease coverage by 0.44%.
The diff coverage is 11.34%.

@@            Coverage Diff             @@
##           master     #307      +/-   ##
==========================================
- Coverage    28.3%   27.86%   -0.45%     
==========================================
  Files          63       63              
  Lines        5949     6280     +331     
==========================================
+ Hits         1684     1750      +66     
- Misses       4265     4530     +265
# [Codecov](https://codecov.io/gh/Plume-org/Plume/pull/307?src=pr&el=h1) Report > Merging [#307](https://codecov.io/gh/Plume-org/Plume/pull/307?src=pr&el=desc) into [master](https://codecov.io/gh/Plume-org/Plume/commit/fdfeeed6d9fa9dec92862366a8b646eb27d17422?src=pr&el=desc) will **decrease** coverage by `0.44%`. > The diff coverage is `11.34%`. ```diff @@ Coverage Diff @@ ## master #307 +/- ## ========================================== - Coverage 28.3% 27.86% -0.45% ========================================== Files 63 63 Lines 5949 6280 +331 ========================================== + Hits 1684 1750 +66 - Misses 4265 4530 +265 ```
elegaanz reageerde 5 jaren geleden (Gemigreerd van github.com)

Ah, yes, I should write tests…

Ah, yes, I should write tests…
elegaanz reageerde 5 jaren geleden (Gemigreerd van github.com)

I think I will wait for #310 to be merged before adding tests, as it would probably create conflicts otherwise…

I think I will wait for #310 to be merged before adding tests, as it would probably create conflicts otherwise…
trinity-1686a beoordeeld 5 jaren geleden
Eigenaar

anything after let conn = ... should be surrounded by a big

conn.test_transaction::<_, (), _>(|| { 
    [...]
    Ok(())
})

to prevent any side effect

anything after `let conn = ...` should be surrounded by a big ```rust conn.test_transaction::<_, (), _>(|| { [...] Ok(()) }) ``` to prevent any side effect
Eigenaar

Is this still ready for review? If so you should fix the conflict with master

Is this still ready for review? If so you should fix the conflict with master
trinity-1686a beoordeeld 5 jaren geleden
trinity-1686a heeft deze veranderingen 5 jaren geleden goedgekeurd

Reviewers

trinity-1686a heeft deze veranderingen 5 jaren geleden goedgekeurd
De pull request is samengevoegd als 4ec2480f50.
Je kunt ook command line instructies bekijken.

Stap 1:

Vanuit het project, check een branch uit en test de veranderingen.
git checkout -b post-api master
git pull origin post-api

Stap 2:

Voeg de wijzigingen samen en update ze op Forgejo.
git checkout master
git merge --no-ff post-api
git push origin master
Log in om deel te nemen aan deze discussie.
Geen beoordelaars
Geen mijlpaal
Niet toegewezen
2 deelnemers
Notificaties
Vervaldatum
De deadline is ongeldig of buiten bereik. Gebruik het formaat 'jjjj-mm-dd'.

Geen vervaldatum ingesteld.

Afhankelijkheden

Geen afhankelijkheden ingesteld.

Referentie: Plume/Plume#307
Laden…
Er is nog geen inhoud.