Inconsistency in `DatasetVersionInfo` model? (Date vs string)

Đang mở Phù hợp với người mới
#476 4 bình luận 0 reaction 0 người được giao Xem trên GitHub

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

Đánh giá

Độ khó
2/5
Thời gian dự kiến
1-3 giờ
Mức phù hợp với người mới
68/100
Loại issue
Lỗi
Độ rõ ràng
Khá rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
typescript
Lĩnh vực
api, backend-api-design

Hướng nghiên cứu

Bắt đầu với định nghĩa DatasetVersionInfo được tham chiếu trong src/datasets/domain/models/Dataset.ts và phần chuyển đổi trong src/datasets/infra/repositories/transformers/datasetTransformers.ts. So sánh cách releaseTime, createTime và lastUpdateTime được biểu diễn so với response của Dataverse API. Được xem là hoàn tất khi model và transformer sử dụng cách biểu diễn timestamp có chủ đích và nhất quán cho cả ba trường.

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

Mô tả

I noticed that in the DatasetVersionInfo model, the timestamps use inconsistent types:

https://github.com/IQSS/dataverse-client-javascript/blob/0043dddf5e7d9358d0dcac176cb8c0f325aab78b/src/datasets/domain/models/Dataset.ts#L25-L32

releaseTime and createTime are represented as Date objects, while lastUpdateTime is represented as an ISO 8601 string and not transformed into a Date object:

https://github.com/IQSS/dataverse-client-javascript/blob/0043dddf5e7d9358d0dcac176cb8c0f325aab78b/src/datasets/infra/repositories/transformers/datasetTransformers.ts#L237-L239

The return values from the Dataverse API for retrieving a dataset version are all consistently ISO 8601 strings:

Image

I was wondering what the reason for this is, or whether this inconsistency should be cleaned up? (If so, I'd be open to opening a PR for that)

Ngôn ngữ chính
TypeScript
Star
20
Fork
11
Merge trung bình
2 ngày 5 giờ
Pull request đã merge (30 ngày)
1

Hướng dẫn đóng góp

Mở hướng dẫn đóng góp

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 IQSS/dataverse-client-javascript

Tất cả issue của IQSS/dataverse-client-javascript

Issue tương tự

Thêm issue về TypeScript

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.