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

Fix ENOENT: no such file or directory #910

Merged
merged 13 commits into from
Aug 7, 2023
Merged

Fix ENOENT: no such file or directory #910

merged 13 commits into from
Aug 7, 2023

Conversation

echaidemenos
Copy link
Collaborator

No description provided.

Copy link

@pullrequest pullrequest bot left a comment

Choose a reason for hiding this comment

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

⚠️ This pull request was not sent to the PullRequest network because the pull request is a draft.

@echaidemenos
Copy link
Collaborator Author

I created a pull request to see if tests on github actions would pass without littering the main PR

@echaidemenos echaidemenos marked this pull request as ready for review August 7, 2023 08:03
Copy link

@pullrequest pullrequest bot left a comment

Choose a reason for hiding this comment

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

✅ This pull request was sent to the PullRequest network.


Check the status or cancel PullRequest code review here.

Copy link

@pullrequest pullrequest bot left a comment

Choose a reason for hiding this comment

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

PullRequest Breakdown

Reviewable lines of change

+ 109
- 138

95% TypeScript (tests)
3% TypeScript
2% JSON

Generated lines of change

+ 54
- 45

Type of change

Fix - These changes are likely to be fixing a bug or issue.
1 Message
📚 It looks like the description for this pull request is either blank or very short. Adding a high-level summary will help our reviewers provide better feedback. Feel free to include questions for PullRequest reviewers and make specific feedback requests.

Copy link

@pullrequest pullrequest bot left a comment

Choose a reason for hiding this comment

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

✖️ This code review was cancelled. See Details

@echaidemenos echaidemenos merged commit 357ab53 into multi-uploads Aug 7, 2023
2 checks passed
@echaidemenos echaidemenos deleted the temp branch August 7, 2023 08:05
echaidemenos added a commit that referenced this pull request Aug 28, 2023
* Allow multiple sites uploads

* Add web page for uploads. Add aqualink_sensor_type token

* Add surveyPoint to dataUploads sites relation. Fix uploads history table

* Fix console warnings

* Add AFFECTED SITE IDS column

* Show survey points for each site

* Add web test

* Add tests

* Add .gitkeep to ./upload

* Revert "Add .gitkeep to ./upload"

This reverts commit 08778cc.

* Fix styles in migration

* Resovle PR comments

* Update snapshots

* Fix ENOENT: no such file or directory (#910)

* Disable errored test and refactor others

* Allow super admins to upload data for any site with the new endpoint

* Update test command flags

* Add single sourceType for whole file upload

* Resolve PR comments

* Fix extra space

* Update huiConfig.ts

* Update index.test.tsx.snap

* Update index.test.tsx.snap

* Update migration to migrate data as well

* Split sheet uploads header rules

---------

Co-authored-by: Eric Boucher <[email protected]>
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