Plume

Panic less and with better messages #281

Integrado

Plume_migration_agent integrou 2 cometimento(s) do ramo panic-refactor no ramo master

trinity-1686a comentou

Proprietário(a)

Fix #21
Change most unwrap to expect, and don't panic when it can be avoided. Also return better http codes, i.e. 400 and 404, when appropriate.
The trait rocker::request::FromParam should probably be implemented for Post, Blog, Media, User and Tag. Then we wouldn't need to return Options whenever an entity does not exist, and just let Rocket do the work for us

Fix #21 Change most unwrap to expect, and don't panic when it can be avoided. Also return better http codes, i.e. 400 and 404, when appropriate. The trait `rocker::request::FromParam` should probably be implemented for Post, Blog, Media, User and Tag. Then we wouldn't need to return `Option`s whenever an entity does not exist, and just let Rocket do the work for us

👍 1 ❤️ 1

elegaanz (Migrado de github.com) aprovou estas modificações

elegaanz (Migrado de github.com) deixou um comentário

Thanks a lot!

elegaanz comentou (Migrado de github.com)

And yes, implementing FromParams for these types would be a good idea.

trinity-1686a comentou

Remetente

Proprietário(a)

After trying to, I guess it might not be possible, because we need a db connection, and I don't think there is a way to access managed state from from_param (it only takes a &RawStr as parameters). Nonetheless I'll ask on Matrix

elegaanz comentou (Migrado de github.com)

Maybe with with FromRequest and Request::get_param then? But it may force us to duplicate parameters in the routes signatures…

Maybe with with `FromRequest` and `Request::get_param` then? But it may force us to duplicate parameters in the routes signatures…

trinity-1686a comentou

Remetente

Proprietário(a)

If we end up doing manual routing it may reduce code duplication a bit, but it will add a lot of complexity, and we won't be able to use rocket's code generation if we do so. I'll see what answer I get from Matrix, but what I said earlier might very well be a false good idea

Revisores

A integração foi executada no cometimento 879fde81ba.

Também pode ver as instruções para a linha de comandos.

Passo 1:

No seu repositório, crie um novo ramo e teste as modificações.

git checkout -b panic-refactor master

git pull origin panic-refactor

Passo 2:

Integre as modificações e envie para o Forgejo.

git checkout master

git merge --no-ff panic-refactor

git push origin master

Inicie a sessão para participar neste diálogo.

Sem revisores

Sem etapa

Sem encarregados

2 Participantes

Notificações

Data de vencimento

A data de vencimento é inválida ou está fora do intervalo permitido. Por favor, use o formato 'aaaa-mm-dd'.

Sem data de vencimento definida.

Dependências

Não estão definidas dependências.

Carregando…

Ainda não há conteúdo.

Rótulos Etapas

Panic less and with better messages #281

Revisores

Passo 1:

Passo 2: