Skip to content
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: store FFD override #220

Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open

feat: store FFD override #220

wants to merge 5 commits into from

Conversation

mathcolo
Copy link
Collaborator

@mathcolo mathcolo commented Jan 8, 2025

Asana Task: 📐 If Orbit FFD expiry is overruled, store that and export it

  • Rename bypass key in API request to override to match Glides
  • Add new override column
  • Save override to database if it passes some rudimentary checks
  • Add data to export CSV

Checklist

  • Tests:
    • (x) Has tests
    • ( ) Doesn't need tests
    • ( ) Tests deferred (with justification)
  • Product/Design sign off:
    • (x) Okayed the plan for the feature (e.g. the design files, or the Asana task)
    • ( ) Reviewed the feature as implemented (e.g. on dev-green, or saw screenshots)
    • ( ) No review needed

@mathcolo mathcolo requested a review from a team as a code owner January 8, 2025 23:12
Comment on lines +38 to +43
|> DateTime.from_iso8601()

datetime
|> DateTime.shift_zone!("America/New_York")
|> DateTime.to_date()
|> Date.to_iso8601()
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Heads up- this is a little messy because the expires is stored in the jsonb column as an ISO timestamp, but I want to provide Excel users with an Excel-compatible date in YYYY-MM-DD. (Yes, this is lossy)

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

Successfully merging this pull request may close these issues.

1 participant