dataiku / dataiku/dataiku-api-client-python
Client list users method have attribute wrongly spell
Chưa có ai nhận issue này.
- Ngôn ngữ chính
- Python
- Star
- 44
- Fork
- 29
- Merge trung bình
- 3 giờ 34 phút
- Pull request đã merge (30 ngày)
- 2
Mô tả
See I wanted to know all the users which are currently active.
Problem with the current code is, list_users() returns list of dictionary. And each dictionary has attribute activeWebSocketSesssions, ideally this should be activeWebSocketSessions.
Note - I wanted to make that change using the pull request, but I did not find this attribute in the current codebase, may be because you guys did not revel it, may be not. Aways this is my finding, that will help make this product a bit better.
def list_active_users(self):
"""
Get all the active users who currently login to the DSS
This method takes nothing and returns name of the users who are actively working now
Returns:
string: calculate all currently active users
"""
user_list = []
dss_users = self.client.list_users()
for user in dss_users:
if user["activeWebSocketSesssions"] != 0:
user_list.append(user["displayName"])
return user_list
Hướng dẫn đóng góp
Chưa lập chỉ mục được hướng dẫn đóng góp cho kho mã nguồn này
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- 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.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Hướng nghiên cứu
Bắt đầu từ điểm vào API client list_users() và kiểm tra cách viết của thuộc tính phiên WebSocket đang hoạt động trong các dictionary người dùng được trả về. Công việc được xem là hoàn tất khi key được trả về sử dụng nhất quán activeWebSocketSessions và cách sử dụng list_active_users() được minh họa có thể xác định những người dùng đang hoạt động mà không phụ thuộc vào key bị viết sai.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Đánh giá
- Công nghệ
- python
- Lĩnh vực
- api
- Loại issue
- Lỗi
- Độ khó
- 2/5
- Thời gian dự kiến
- 1-3 giờ
- Mức độ hoạt động
- Đình trệ
- Độ rõ ràng
- Khá rõ ràng
- Mức phù hợp với người mới
- 48/100