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

Event Upsert #75

Open
Tracked by #90
Thejagd27 opened this issue May 2, 2023 · 1 comment · May be fixed by #101
Open
Tracked by #90

Event Upsert #75

Thejagd27 opened this issue May 2, 2023 · 1 comment · May be fixed by #101
Labels
enhancement New feature or request pending-review issues which have corresponding PRs which have a pending review/merge from the maintainer.

Comments

@Thejagd27
Copy link

@ChakshuGautam Please review the approach and suggest if any changes required
Issue with Event Upsert

Previously Discussed Approach-> When an event file is uploaded, check if there is already a event file for today. If there is a file then get the name of the system file for that event Create a CSV file with same name and upload. So that the old data file will be replaced with updated data
But with the above approach, if the state team wants to upload new data second time in a day , then their old data will be replaced so will loose the previously uploaded data as it overrides the data.
 
Proposed Approach for Event Upsert

Will create a new API as PUT /ingestion/event
Using this API they can update the data
And for uploading new data they can use the existing /ingestion/new_programs API for uploading more than once in a day

@techsavvyash
Copy link
Collaborator

implemented in #101

@techsavvyash techsavvyash added enhancement New feature or request pending-review issues which have corresponding PRs which have a pending review/merge from the maintainer. labels Oct 7, 2023
@techsavvyash techsavvyash linked a pull request Oct 7, 2023 that will close this issue
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request pending-review issues which have corresponding PRs which have a pending review/merge from the maintainer.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants