Merge pull request #1 from yannick1691/dev

Added confirm delete group and changed doel from type date to type text
zoeyfrisart · Nov 6, 2017 · a3347bb · a3347bb
2 parents 335f119 + 35af81d
commit a3347bb
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 6 deletions.
diff --git a/components/groups/index.js b/components/groups/index.js
@@ -86,9 +86,12 @@ export default class BoardGroup extends Component {
  this.props.updateGroup(this.props.index, updatedGroup);
  };
 
- handleDeleteGroupItems(){
- base.remove(`/items/${this.props.index}`);
- this.props.removeGroup(this.props.index);
+ handleDeleteGroupItems() {
+ let r = confirm(`You are about to delete ${this.props.details.name}. \nThis can't be undone! \nAre you sure you want to delete the board ${this.props.details.name}`);
+ if (r === true) {
+ base.remove(`/items/${this.props.index}`);
+ this.props.removeGroup(this.props.index);
+ }
  }
 
  render() {

diff --git a/components/item/index.js b/components/item/index.js
@@ -105,9 +105,9 @@ export default class Item extends Component {
  class={style.deadline}
  name="finishGoal"
  onChange={(e) => this.handleChange(e, this.props.index)}
- type="date"
+ type="text"
  value={details.finishGoal}
- placeholder="dd/mm/yyyy"
+ placeholder="dd/mm/yy"
  />
  </div>
  </td>
@@ -120,7 +120,7 @@ export default class Item extends Component {
  onChange={(e) => this.handleChange(e, this.props.index)}
  type="date"
  value={details.deadline}
- placeholder="dd/mm/yyyy"
+ placeholder="dd/mm/yy"
  />
  </div>
  </td>