Skip to content
This repository has been archived by the owner on Sep 8, 2020. It is now read-only.

Expired Data Detection #22

Open
nareddyt opened this issue Mar 13, 2018 · 5 comments
Open

Expired Data Detection #22

nareddyt opened this issue Mar 13, 2018 · 5 comments
Labels
enhancement New feature or request good first issue Good for beginners

Comments

@nareddyt
Copy link
Owner

In data/deals.json, some objects contain an expiry date. We can check this expiry date against the current date to infer when our data is stale, signalling that we MUST update the data.

So it would be nice to have a script that checks this data. Perhaps a new node script in tools/ that we can run via an npm script in package.json.

It would be awesome if we could integrate this into our CircleCI workflow. In case someone publishes the extension with expired data, CircleCI could throw an error and prevent it from being published!

Note: Some deals have no expiry date, so those may expire at any time. Those deals are out-of-scope for this issue, so feel free to ignore them.

@nareddyt nareddyt added good first issue Good for beginners data-update Related to updating deals.json or cashbacks.json enhancement New feature or request and removed data-update Related to updating deals.json or cashbacks.json labels Mar 13, 2018
@arwalokhandwala
Copy link

arwalokhandwala commented May 21, 2018

Hi @nareddyt I would like to work on this issue, this would be my first time contribution.

@nareddyt
Copy link
Owner Author

Sounds good! Let me know if you have any questions. I'm a little busy this week but I'd be glad to clarify anything that is unclear.

You can view some example data in the data folder. Note that only deals can have expiry dates, but not all deals have expiry dates. Take a look at data/deals/schema.json for an example of what values the expiry date can take on!

Here's a brief overview of what continuous integration is if you choose to integrate it into out CircleCI workflow.

@jmoore914
Copy link
Contributor

Hi @nareddyt, it looks like there hasn't been any work on this is a few months so I can take a stab at it!

@jmoore914
Copy link
Contributor

Here's a link to the PR:
#82 (comment)

This is my first contribution so sorry if I didn't follow proper protocol!

@nareddyt
Copy link
Owner Author

Thanks, that works! I'll take a look at it soon

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request good first issue Good for beginners
Projects
None yet
Development

No branches or pull requests

3 participants