allow for pm times to be entered

lumjjb · Aug 9, 2021 · 23c839f · 23c839f
1 parent 881a811
commit 23c839f
Showing 1 changed file with 6 additions and 5 deletions.
diff --git a/tornjak-frontend/src/components/entry-create.js b/tornjak-frontend/src/components/entry-create.js
@@ -216,15 +216,16 @@ class CreateEntry extends Component {
  });
  }
 
- onChangeExpiryOption(e) { //TODO try date=date
+ onChangeExpiryOption(e) {
  this.setState({
  expiresAt: 0,
  expiryOption: e.selectedItem 
  });
  }
 
  isValidJSTime(seconds) {
- return seconds > 0 && seconds <= 8640000000000 //TODO clean
+ const JSMaxSafeTime = 8640000000000
+ return seconds > 0 && seconds <= JSMaxSafeTime
  }
 
  expiryTimeUpdate(seconds) {
@@ -245,7 +246,8 @@ class CreateEntry extends Component {
  this.expiryTimeUpdate(seconds)
  }
 
- isValidDate(d) {
+ // TODO some odd behavior with dates like February 33 exists
+ isValidDate(d) { // date is successfully translated 
  return d instanceof Date && isFinite(d)
  }
 
@@ -702,16 +704,15 @@ class CreateEntry extends Component {
  placeholder="mm/dd/yyyy"
  labelText="Enter a date"
  onChange={this.onChangeExpiresAtDate}
- required
  />
  </DatePicker> 
  </div>
  <div className="expiryOption-entry">
  <TimePicker
  labelText="Enter local time (15:45)"
  placeholder="hh:mm"
+ pattern={["^([0-1]\\d|2[0-3]):[0-5]\\d:[0-5]\\d$"]} // Allows 
  onChange={this.onChangeExpiresAtTime}
- required
  >
  </TimePicker>
  </div>