Hacktoberfest 2026: những issue maintainer đã đánh dấu cho tháng Mười, đang mở và phù hợp người mới. Xem issue Hacktoberfest

Hide the per-item credit preference field from non-admins once profiles are self-service

Đang mở
  • Có gì không ổn?
#2,261 0 bình luận 0 reaction 0 người được giao Xem trên GitHub

Maintainer thường phản hồi trong vòng 1 ngày

Chưa có ai nhận issue này.

Đánh giá

Độ khó
4/5
Thời gian dự kiến
3-5 ngày
Mức phù hợp với người mới
48/100
Loại issue
Tái cấu trúc
Độ rõ ràng
Khá rõ ràng
Mức độ hoạt động
Ít trao đổi
Công nghệ
rails, ruby
Lĩnh vực
authorization, full-stack

Hướng nghiên cứu

Bắt đầu bằng cách đọc app/views/shared/_author_credit_status.html.erb và app/views/shared/_author_credit_preview.html.erb, sau đó lần theo ba biểu mẫu ý tưởng dành cho người gửi và các controller của chúng. Xác nhận điều kiện tiên quyết để tự chỉnh sửa hồ sơ, hạn chế hoặc xóa trường tùy chọn và params, cập nhật hướng dẫn, đồng thời xác định liệu phần tùy chọn có còn trên /author_credit_divergences hay không.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

Interim state shipped in #2093.

Why the field is there now

Credits render from the credited person's profile (Person#display_name_preference plus anonymous_contributions). Facilitators cannot edit their own profile — PersonPolicy#edit? and #update? are admin? — so the submission forms still collect a per-item credit preference as a way to record what the submitter wants.

That stored value does not drive display. It surfaces on /author_credit_divergences as a discrepancy, and an admin applies it to the person's profile so it covers everything they share. The edit forms already warn when a record diverges, linking to that page.

What to do here

Once facilitators can edit their own display preferences:

  • Hide author_credit_preference on the three submitter-facing idea forms (story_ideas, workshop_ideas, workshop_variation_ideas) unless allowed_to?(:manage?, ...) — the field lives in app/views/shared/_author_credit_status.html.erb
  • Drop :author_credit_preference from those three controllers' strong params, or keep it admin-only
  • Point submitters at their own profile instead — replace the "an admin applies it to your profile" copy in app/views/shared/_author_credit_preview.html.erb
  • Revisit whether the preference section of the divergences page is still needed, or only the three sections about a missing author_id

Related

  • Self-service profile editing is the blocker: PersonPolicy#edit? / #update? are admin-only, and person_params permits notes, filemaker_code, created_by_id and nested user_attributes[:inactive], so it needs its own narrow whitelist.
  • The per-item anonymous override is a separate legacy path the divergences page also drains.

🤖 From Claude: filed while shipping the interim step in #2093.

Ngôn ngữ chính
Ruby
Star
15
Fork
28
Merge trung bình
1 ngày 12 phút
Pull request đã merge (30 ngày)
262

Chuẩn bị môi trường

Mở trong Codespaces

Khởi chạy dev container của dự án ngay trên trình duyệt, bằng tài khoản GitHub của bạn.

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của rubyforgood/awbw

Tất cả issue của rubyforgood/awbw

Issue tương tự

Thêm issue về Ruby

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.