You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository has been archived by the owner on Jun 8, 2020. It is now read-only.
A couple of notes from the feedback on the previous iteration:
Rachael liked the calendar select for setting expiration times. If we can find something that works well with mobile, it seems like that would be her preferred input method.
She liked that notes defaulted to expiring after 5 hours, and liked having some quick select options for expiration timers. (I'm not 100% sure about this one, my notes are somewhat vague here. Please correct this if you remember her saying differently)
The text was updated successfully, but these errors were encountered:
Add addDate and expireDate fields to the Note type in the backend, and make status an enumeration rather than a boolean.. Also fix a small breaking bug in note.service.spec. For issue #11.
Change the expireDate to a Date, to match the client and improve ease of use; change the status to an enumeration, replacing the boolean "posted" to allow for drafts in the future; and improve editing notes in order to allow the expiration date and status to be edited. For issue #11.
Add date and time boxes which should create an expireDate when filled; however, tests are unable to run. I'm not entirely sure why, and am committing just to save progress. Issue #11.
Sign up for freeto subscribe to this conversation on GitHub.
Already have an account?
Sign in.
A couple of notes from the feedback on the previous iteration:
Rachael liked the calendar select for setting expiration times. If we can find something that works well with mobile, it seems like that would be her preferred input method.
She liked that notes defaulted to expiring after 5 hours, and liked having some quick select options for expiration timers. (I'm not 100% sure about this one, my notes are somewhat vague here. Please correct this if you remember her saying differently)
The text was updated successfully, but these errors were encountered: