Skip to content

Improve Migrations Stablity - #7

Merged
christyjacob4 merged 42 commits into
mainfrom
feat-improve-features
Aug 9, 2023
Merged

Improve Migrations Stablity#7
christyjacob4 merged 42 commits into
mainfrom
feat-improve-features

Conversation

@PineappleIOnic

@PineappleIOnicPineappleIOnic commented Jul 18, 2023•
edited
Loading

Copy link
Copy Markdown
Contributor
  • Add size and version to reports to allow more data for frontend
  • Implement 3 step file process to allow for files that don't fit the bucket's restrictions to still be transferred as intended
  • Fix File Transfers
  • Implement Tests for NHost and Supabase

@PineappleIOnic
Continue Work
304d426
@PineappleIOnic
Merge branch 'main' into feat-implement-tests
14e6491
@PineappleIOnic
Update Appwrite.php
cc73348
@PineappleIOnic
Continue work on tests
593ffb6
@PineappleIOnic
Merge branch 'feat-improve-reports' into feat-improve-features
dbdce49
@PineappleIOnic
Merge branch 'feat-implement-tests' into feat-improve-features
1f87a8c
@PineappleIOnic
Update Transfer.php
e19cf37
@PineappleIOnic
Wrap getId from ?string to string
43bd801
@PineappleIOnic
Keep working on Transfers
d22c31a
@PineappleIOnic
Add size for NHost and Supabase
e4f3d43
@PineappleIOnic
Fix File Transfers
a30b2b2
@PineappleIOnic
Add 3 step file process
813901b
@PineappleIOnic
Add 3 step storage transfers
536e6a3
@PineappleIOnic
Fix team memberships
0eedfb0
@PineappleIOnic
Bring back deployment transfers for versions >1.4.0
48f21a9
@PineappleIOnic
Fix functions and subcollections
c7bae26
@PineappleIOnic
Merge branch 'main' into feat-improve-features
54a4283
@PineappleIOnic
Improve Status Counters to use last report to know pending
020f52a
@PineappleIOnic
Continue work on tests
41540eb
@PineappleIOnic
Fix Files
947a2e6
@PineappleIOnic
Add Tests for Supabase and NHost
5d9f32c
@PineappleIOnic
Add Workflow
3557317
@PineappleIOnic
Update tests.yml
c45e82c
@PineappleIOnic
Update Dockerfile
f5f7a3a
@PineappleIOnic
Fix report merge
92fee96
@PineappleIOnic
Finish Tests for NHost and Supabase and Run Linter
ff61281
@PineappleIOnic
Continue work on tests
e7e222f
@PineappleIOnic
Continue work on tests
01d0d4d
@PineappleIOnic
Update playground.php
86afe1e
@PineappleIOnic
Update playground.php
7362d0e
@github-actionsgithub-actionsBot mentioned this pull request Aug 1, 2023
Closed
christyjacob4
christyjacob4 reviewed Aug 1, 2023
View reviewed changes
Comment threadtests/Transfer/resources/supabase/aaa.sql
christyjacob4
christyjacob4 reviewed Aug 1, 2023
View reviewed changes
Comment threadtests/Transfer/resources/nhost/aaa.sql
christyjacob4
christyjacob4 reviewed Aug 1, 2023
View reviewed changes
Comment threadsrc/Transfer/Resources/Auth/Team.php Outdated
christyjacob4
christyjacob4 reviewed Aug 1, 2023
View reviewed changes
Comment threadsrc/Transfer/Resources/Auth/User.php Outdated
christyjacob4
christyjacob4 reviewed Aug 1, 2023
View reviewed changes
Comment threadsrc/Transfer/Resources/Database/Collection.php Outdated
christyjacob4
christyjacob4 reviewed Aug 1, 2023
View reviewed changes
Comment threadsrc/Transfer/Resources/Database/Database.php Outdated
christyjacob4
christyjacob4 reviewed Aug 1, 2023
View reviewed changes
Comment threadtests/Transfer/E2E/Adapters/Mock.php
christyjacob4
christyjacob4 requested changes Aug 1, 2023
View reviewed changes

@christyjacob4christyjacob4 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good. Just some minor comments 😄

vermakhushboo
vermakhushboo reviewed Aug 1, 2023
View reviewed changes
Comment threadDockerfile
WORKDIR /app
RUN git clone https://github.com/PineappleIOnic/mockoon.git .
RUN apk add --no-cache git
RUN git clone https://github.com/PineappleIOnic/mockoon.git

@vermakhushboovermakhushbooAug 1, 2023•
edited
Loading

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Is this added only for testing or will it also be used in prod? Do we want to use the forked repo or the actual one?

@PineappleIOnicPineappleIOnicAug 3, 2023

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

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

Mockoon currently doesn't support range headers, I have a PR in the works to bring this feature over to their repo but currently I don't have the time to write tests for them so for now we are using my fork and will switch once I have more time to finish their tests and merge it into their repo

vermakhushboo
vermakhushboo reviewed Aug 1, 2023
View reviewed changes
Comment threadsrc/Transfer/Destinations/Appwrite.php Outdated
abnegate
abnegate reviewed Aug 2, 2023
View reviewed changes
Comment threadplayground.php Outdated
Comment threadsrc/Transfer/Cache.php Outdated
Comment threadsrc/Transfer/Destinations/Appwrite.php Outdated
Comment threadsrc/Transfer/Destinations/Appwrite.php
Comment threadsrc/Transfer/Destinations/Appwrite.php Outdated
Comment threadsrc/Transfer/Sources/Supabase.php
Comment threadtests/Transfer/E2E/Sources/Base.php Outdated
Comment threadtests/Transfer/E2E/Sources/Base.php Outdated
Comment threadtests/Transfer/E2E/Sources/Base.php
Comment threadtests/Transfer/E2E/Adapters/Mock.php Outdated
PineappleIOnicand others added 5 commits August 4, 2023 00:14
@PineappleIOnic@abnegate
Update src/Transfer/Destinations/Appwrite.php
9d4b0a0
Co-authored-by: Jake Barnby <jakeb994@gmail.com>
@PineappleIOnic@abnegate
Update src/Transfer/Sources/Appwrite.php
4485e8a
Co-authored-by: Jake Barnby <jakeb994@gmail.com>
@PineappleIOnic@abnegate
Update src/Transfer/Cache.php
9879e86
Co-authored-by: Jake Barnby <jakeb994@gmail.com>
@PineappleIOnic
Continue work on suggestions
effcf8e
@PineappleIOnic
Continue work on suggestions
ba56752
@christyjacob4
christyjacob4 merged commit 4535b09 into mainAug 9, 2023
@github-actionsgithub-actionsBot mentioned this pull request Sep 1, 2023
Closed
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@christyjacob4christyjacob4christyjacob4 approved these changes
@abnegateabnegateAwaiting requested review from abnegate
+1 more reviewer
@vermakhushboovermakhushboovermakhushboo left review comments
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@PineappleIOnic@abnegate@christyjacob4@vermakhushboo