Restrict uploads

고맙습니다, 작동합니다

이제 이 컴포넌트를 구독자 그룹으로 제한할 수 있을까요 :stuck_out_tongue:

제가 직접 할 수 있는 유일한 방법은 Trust Level 3에 도달할 수 없도록 일수 등을 늘리는 것입니다. 그리고 사람들이 구독할 때 수동으로 추가하는 방식이죠. 그룹별로 이 부분을 수정할 수 없다면 아마도 이 방법을 따를 것 같습니다.

2개의 좋아요

업로드 제한 기능이 이제 새 게시물이나 답변에서 아예 업로드를 차단하고 있습니다.

1개의 좋아요

맞아요, 죄송합니다.

this._super()가 실제로 코어에서 사용자가 이미지를 업로드할 수 있는지 여부를 보고하지 않는 것 같습니다. 기존 컴포넌트에서 재사용했는데, 어제 이미 플레이스홀더 로직에도 문제가 발생한다는 점을 알아차렸습니다.

지금 이 문제를 더 깊이 파고들 시간이 없습니다.

오늘 테스트 중 발견한 점이 하나 있습니다: api.addComposerUploadHandler(["*"]에서 "*"를 사용하면 업로드 핸들러 매칭에 사용되는 정규식이 깨집니다.

1개의 좋아요

이것은 코어에 포함되는 것이 좋을 것 같습니다. 제 개인적인 의견이지만, 신뢰 수준, 그룹 또는 개별 사용자별로 업로드를 제어할 수 있는 방법은 매우 유용할 것입니다.

2개의 좋아요

이 계산된 디스크립터를 올바르게 오버라이드하려면 서브클래스 메서드를 사용해야 한다고 생각합니다.

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

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

수정:

PR을 만들었습니다:

4개의 좋아요

구성 요소의 제한 방식을 신뢰 수준 기반에서 그룹 기반으로 전환하는 풀 리퀘스트를 생성했습니다:

9개의 좋아요

@tshenry 컴포넌트를 그룹 기반으로 작동하도록 변경하는 데 관심이 있으신가요?

테마의 그룹 설정에 대한 코어의 변경 사항을 반영하기 위해 이제 변경 사항을 조정해야 할 가능성이 높습니다. 하지만 원치 않으시는 경우, 업데이트에 시간을 들이지 않아도 됩니다.

1개의 좋아요

@Moin 그룹 설정 변경 사항에 대해 언급하신 것으로 추정되는 부분에 대해 PR에 댓글을 남겼습니다. 이 작업에 계속 관심이 있으시다면 업데이트를 푸시해 주십시오. 현재 테스트 사이트에 PR 브랜치를 설정해 두었으므로, 이 작업을 진행하는 데 도움을 드릴 수 있을 것입니다.

2개의 좋아요

@Moin 님의 이 컴포넌트에 대한 훌륭한 업데이트가 병합되었습니다 :tada: Moin 님, 감사합니다!

@Arkshine 님과 @Moin 님이 올해 가져온 모든 개선 사항을 반영하도록 이 주제의 원포스터(OP)를 곧 업데이트하겠습니다.

4개의 좋아요