Skip to content
This repository has been archived by the owner on Dec 7, 2024. It is now read-only.

str-time may lead to confusion on this field name #30

Open
naomijub opened this issue Jun 15, 2023 · 0 comments
Open

str-time may lead to confusion on this field name #30

naomijub opened this issue Jun 15, 2023 · 0 comments
Labels
bug Something isn't working good first issue Good for newcomers

Comments

@naomijub
Copy link
Owner

str-time may lead us to believe that this is a string, should it be called something like optional-time? Also, this is assuming that the time will be optional, correct? But we can change later if we see that some API requires a tie param o/

Originally posted by @pfeodrippe in #29 (comment)

Suggested names might be str-proto-time or proto-str-time
@naomijub naomijub added bug Something isn't working good first issue Good for newcomers labels Jun 15, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
bug Something isn't working good first issue Good for newcomers
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@naomijub