bubblev
/
bubble
9
0
Derivar 0
Código Questões 0 Pedidos de integração 0 Lançamentos 130 Wiki Trabalho
Etiquetas Etapas
Novo pedido de integração

#53 Fully forbid network owner deletion

Integração executada
jonathan integrou 7 cometimentos de %!<(string=kris/forbid_network_owner_deletion)/code> em master há 4 anos
Diálogo 0 Cometimentos 7 Ficheiros modificados 3
kris comentado há 4 anos

@jonathan please review

Also included fix for Jenkins builds when utils submodules’ links are changed in PRs.

@jonathan please review Also included fix for Jenkins builds when utils submodules' links are changed in PRs.
jonathan foi atribuída por kris há 4 anos
jonathan alterações solicitadas há 4 anos
jonathan deixou um comentário

some minor changes requested

bubble-server/src/main/java/bubble/dao/account/AccountDAO.java
@@ -347,2 +349,4 @@
}

@Override
public int bulkDeleteWhere(@NonNull final String whereClause, @NonNull final Map<String, Object> parameters) {
jonathan comentado há 4 anos

Please put @Override on the same line as the method declaration.

Please put `@Override` on the same line as the method declaration.
bubble-server/src/main/java/bubble/dao/account/AccountDAO.java
@@ -349,0 +355,4 @@
}

@Override
public void delete(@Nullable final Collection<Account> accounts) {
jonathan comentado há 4 anos

Please put @Override on the same line as the method declaration.

Please put `@Override` on the same line as the method declaration.
bubble-server/src/main/java/bubble/service/stream/AppDataCleaner.java
@@ -26,3 +28,3 @@
@Override protected void process() {
try {
final int ct = dataDAO.bulkDeleteWhere("expiration < " + now());
final int ct = dataDAO.bulkDeleteWhere("expiration < " + now(), new HashMap<>());
jonathan comentado há 4 anos

If we keep the original bulkDeleteWhere we don’t need this change.

If we keep the original `bulkDeleteWhere` we don't need this change.
jonathan fechou este pedido de integração há 4 anos
jonathan eliminou o ramo kris/forbid_network_owner_deletion há 4 anos

Revisores

jonathan alterações solicitadas há 4 anos
A integração constante no pedido foi executada como 189eda7d46.
Inicie a sessão para participar neste diálogo.
Revisores
Solicitar avaliação
Sem revisores
 jonathan
Etiquetas
Aplicar etiquetas
Limpar etiquetas
Sem itens
Sem etiquetas
Etapa
Definir etapa
Limpar etapa
Sem itens
Sem etapa
Responsáveis
Definir responsáveis
Limpar responsáveis
Sem responsáveis
 jonathan
2 Participantes
Notificações
Date limite

Sem data limite definida.

Dependências

Este pedido de integração não tem quaisquer dependências, neste momento.

Escrever Pré-visualizar
Carregando…
Cancelar
Guardar
Ainda não há conteúdo.
Eliminar o ramo 'kris/forbid_network_owner_deletion'

Eliminar um ramo é algo permanente. NÃO PODERÁ ser revertido. Quer continuar?

Não
Sim