time_window does not read ISO datetimes in start and end

Đang mở
  • Có gì không ổn?
#1,156 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ó
3/5
Thời gian dự kiến
1-2 ngày
Mức phù hợp với người mới
65/100
Loại issue
Tính năng
Độ rõ ràng
Khá rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
python
Lĩnh vực
backend

Hướng nghiên cứu

Start by locating the rules-engine parsing for time_window and the flag/experiment save validation path, then compare how start/end, start_time/end_time, and date fields are read. Add or update tests around ISO datetime windows, including the now-1h to now+1h case. Done means either start/end interpret ISO datetimes as an absolute window, or saving rejects unsupported values with a message listing accepted forms.

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

Mô tả

enhancement post-launch

A time_window condition whose start/end are ISO datetimes is not read by the rules engine. Measured on 0.28.2 with flag targeting: a window from now-1h to now+1h does not match. Experiment targeting matches it today, but #1145 makes experiments use the same implementation as flags, so after #1145 neither matches.

The engine reads HH:MM times (start/end in the dashboard shape, start_time/end_time natively), days, and start_date/end_date.

Expected: one of these:

  • start/end accept ISO datetimes and mean an absolute window;
  • or the API refuses a time_window value it cannot read when the flag or experiment is saved, with a message naming the accepted forms.

Not needed for launch: after #1145 such a window matches nobody, which is the safe direction; this issue makes it usable.

🤖 Generated with Claude Code

Ngôn ngữ chính
Python
Star
0
Fork
4
Merge trung bình
5 giờ 19 phút
Pull request đã merge (30 ngày)
503

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

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 Controlvar/controlvar

Tất cả issue của Controlvar/controlvar

Issue tương tự

Thêm issue về Python

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.