-
Notifications
You must be signed in to change notification settings - Fork 2
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Feat: Edit Timer! [MVP] 📝 #195
Comments
@LuchoTurtle busy reviewing your PR #196 |
@nelsonic show me tomorrow morning. It should be fixeable. I've been so into Flutter and 99% close to finishing it. It was super worth it though, the other two issues you've assigned to me for the examples should be much easier after I'm done with Sorry for taking this long but I had to do a complete overhaul and added much more info and retained as much as possible.. I digress. No need to send a video. Demo me tomorrow on standup or before and I'll take a look at it 👉😎👉 |
Cool. Will show you IRL or on Standup and we can walk through it. 👌 |
As discussed on standup this morning ... 💬 So we shouldn't require a LMK if this is clear. 🙏 |
@nelsonic didn't get to have the timer editing with stop at |
Great spot! That should absolutely have been a requirement/acceptance criteria in the OP above! 💭 (adding it now!) |
The timer can be updated while it is running. Validation now works properly.
Concerned that we're including quite a few hundred lines of Library/Utility Code for Date/Time Parsing in the
If we need to write our own date/time parsing library this should be captured somewhere as a comment in this issue. |
However, you raise a good point about having to "pay rent" for each line we write. I thought it'd be a good idea to have a small parser util instead of importing a fully-fledged library just to parse strings into |
@LuchoTurtle to be clear, I'm not at all against writing Library Code. We do it all the time, e.g: https://github.com/dwyl/useful The only reason I'm flagging the addition of But if we writing a parser just because we didn't do enough research into the available packages then "That's a Nicht Nicht" 😜 Again, perfectly happy for us to have our own |
Importing a library for a single function in the MVP is fine provided it meets our exact needs. ✅ |
Following on from #190 where you were able to run the
MVP
onlocalhost
🎉 (and update the docs! 🙌)You should feel confident enough in you knowledge of the project and
Phoenix/LiveView
skillsto create a
new
feature that has been sorely lacking from the MVP. ⏳This feature is described in more detail in the issue: dwyl/app#282
But the gist from the MVP's perspective is the following:
Todo
person
enters theedit
mode for anitem
:Should:
timers
for thatitem
:timer.start
ortimer.stop
in theUI
and clicking/tapping theUpdate
button will:timer.start
andtimer.stop
are valid Date + Timei.e:
sanitise
and attempt toparse
the datatimer.stop
is notbefore
thetimer.start
i.e. invalid!i.e: the second timer cannot have
timer.start
set to a date+time earlier than thetimer.stop
of the first timer.We can add fancy UI/UX for timer input later for now, just text is fine for MVP/Alpha!
@LuchoTurtle
new
branch on yourlocalhost
with the tests, update toBUILDIT.md
and code 🆕commit
ting ,git push
and 💻PR
e.g: "PR: Edit Timers Feat: Edit Timer! [MVP] 📝 #195` 🆕in-progress
until you feel it's in a reviewable state. 🧑💻The text was updated successfully, but these errors were encountered: