Restringir uploads

Personalização Componente de tema
25

image
image1202×382 16.5 KB

Obrigado, funcionou

26

Agora podemos fazer este componente restringir-se a grupos de assinantes :stuck_out_tongue:

A única maneira que posso fazer isso sozinho é tornando o Nível de Confiança 3 impossível de ser alcançado, aumentando o número de dias, etc. E então adicionando pessoas manualmente quando elas se inscrevem. O que é muito provavelmente o caminho que seguirei, a menos que eu possa modificar isso por grupo.

2 curtidas
27

Restringir Uploads agora está bloqueando a capacidade de fazer upload de qualquer coisa em novas publicações ou respostas.

1 curtida
28

Isso mesmo, desculpe.

Parece que this._super() na verdade não relata se o usuário tem permissão para enviar imagens no core. Eu o reutilizei do componente antigo e já percebi ontem que ele também causava problemas com a lógica do espaço reservado (placeholder).

Não tenho tempo para investigar isso mais a fundo agora.

Uma coisa que notei ao testar hoje: usar \"*\" em api.addComposerUploadHandler([\"*\"] quebra a expressão regular usada para corresponder aos manipuladores de upload.

1 curtida
29

Eu acho que isso deveria fazer parte do núcleo. Apenas minha humilde opinião, mas uma forma de controlar uploads por nível de confiança, grupo ou até mesmo usuários individuais seria muito útil.

2 curtidas
30

Acredito que você precise usar o método modifyClass para sobrescrever este descritor computado corretamente.

import discourseComputed from "discourse-common/utils/decorators";

api.modifyClass("service:composer", 
  (SuperClass) => class extends SuperClass {
    @discourseComputed
    allowUpload() {
      return super.allowUpload && canUpload;
    }
  }
);

EDIT:

Fiz um PR:

4 curtidas
Returning bumping after editing last post
31

Eu criei um pull request que troca a restrição baseada em nível de confiança do componente para baseada em grupo:

9 curtidas
33

@tshenry você tem interesse em alterar o componente para funcionar com base em grupos?

As alterações provavelmente precisariam ser ajustadas agora para refletir as mudanças no núcleo em relação às configurações de grupos nos temas. Mas, se você não quiser a mudança de qualquer forma, não preciso gastar tempo com uma atualização.

1 curtida
34

@Moin Acabei de deixar um comentário na sua PR sobre o que eu acho que você estava se referindo nas alterações de configuração do grupo. Sinta-se à vontade para enviar uma atualização se ainda tiver interesse em trabalhar nisso. Já configurei a branch da sua PR no meu site de testes, então devo conseguir ajudar a avançar com isso.

2 curtidas
35

A excelente atualização do @Moin para este componente foi mesclada :tada: Obrigado, Moin!

Vou tentar atualizar o autor deste tópico em breve para refletir todas as melhorias que vieram de @Arkshine e @Moin este ano.

4 curtidas