Skip to content
This repository has been archived by the owner on Jan 6, 2021. It is now read-only.

Warning message for overdue applications #87

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

KNWR
Copy link
Contributor

@KNWR KNWR commented Jan 21, 2020

Goal:

If company is in applied AND current date - pitch date >= 6 days then
-- put to the top of the stack
-- have red warning text (as shown)

Notes

  • This isn't quite working yet - is it an issue with how I'm setting createdAt in seeds?
  • Trying to get {this.props.company.createdAt} doesn't print anything

Not directly related to above issue:

  • Could try to use json property in typeformdata on company model and sort by actual application date instead of when the company model was created on our server (would have to convert that to an attribute on the company model to use in sql queries, I believe) ... or could pass it as props and in column.tsx do more in memory sorting, but I reckon that's messy and we want to do less of that as is (shouldn't load whole thing into memory when db query can handle it)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant