Update all deps of all apps#1536
Merged
Merged
Conversation
ytimocin
force-pushed
the
ytimocin/updateDeps
branch
from
7dbf462 to
c4ee1ae
Compare
Signed-off-by: ytimocin <ytimocin@microsoft.com>
ytimocin
force-pushed
the
ytimocin/updateDeps
branch
from
c4ee1ae to
1466a0a
Compare
ytimocin
commented
May 1, 2024
| const [body, setBody] = React.useState(""); | ||
| const [reloadCount, setReloadCount] = React.useState(0); | ||
|
|
||
| const [title, setTitle] = React.useState(""); |
Contributor
Author
There was a problem hiding this comment.
This was unused and it was throwing a warning.
ytimocin
commented
May 1, 2024
| mounted = false; | ||
| } | ||
| }, [reloadCount]) | ||
| }, [body, method, reloadCount, url]) |
Contributor
Author
There was a problem hiding this comment.
This was also giving a warning.
ytimocin
commented
May 1, 2024
| }) | ||
| return result.value as Item | null; | ||
| }); | ||
| return result?.value as Item | null; |
Contributor
Author
There was a problem hiding this comment.
This was throwing an error with the updated deps.
rynowak
reviewed
May 2, 2024
Contributor
rynowak
left a comment
rynowak
left a comment
There was a problem hiding this comment.
The code changes look reasonable. What did you do to test this out?
Contributor
Author
I tested the demo locally and also one of the checks tests demo application with the updated package.json file. |
rynowak
approved these changes
May 2, 2024
kachawla
approved these changes
May 3, 2024
sk593
pushed a commit
that referenced
this pull request
May 23, 2024
Signed-off-by: ytimocin <ytimocin@microsoft.com> Signed-off-by: sk593 <shruthikumar@microsoft.com>
sk593
pushed a commit
that referenced
this pull request
May 23, 2024
Signed-off-by: ytimocin <ytimocin@microsoft.com> Signed-off-by: sk593 <shruthikumar@microsoft.com>
sk593
added a commit
that referenced
this pull request
May 23, 2024
* update gh to ado sync workflow to use service principals (#1363) * Make env configurable (#1046) Signed-off-by: Reshma Abdul Rahim <reshmarahim.abdul@microsoft.com> * Adding test AKS workflow to v0.32 (#1052) * Run functional tests every 2 hours and add another workflow that runs… (#1020) * Run functional tests every 2 hours and add another workflow that runs the tests on AKS instead of k3d Signed-off-by: ytimocin <ytimocin@microsoft.com> * Triggering workflow Signed-off-by: ytimocin <ytimocin@microsoft.com> --------- Signed-off-by: ytimocin <ytimocin@microsoft.com> * Removing the run of the AKS workflow on PRs (#1051) Signed-off-by: ytimocin <ytimocin@microsoft.com> * Update the timeout (30s) for Playwright Signed-off-by: ytimocin <ytimocin@microsoft.com> --------- Signed-off-by: ytimocin <ytimocin@microsoft.com> * Use retry and update eshop playwright tests to wait for the catalog to appear (#1213) Signed-off-by: ytimocin <ytimocin@microsoft.com> * update gh to ado sync workflow to use service principals Signed-off-by: Will Tsai <28876888+willtsai@users.noreply.github.com> --------- Signed-off-by: Reshma Abdul Rahim <reshmarahim.abdul@microsoft.com> Signed-off-by: ytimocin <ytimocin@microsoft.com> Signed-off-by: Will Tsai <28876888+willtsai@users.noreply.github.com> Signed-off-by: Will <28876888+willtsai@users.noreply.github.com> Co-authored-by: Reshma Abdul Rahim <61033581+Reshrahim@users.noreply.github.com> Co-authored-by: Yetkin Timocin <ytimocin@microsoft.com> Signed-off-by: sk593 <shruthikumar@microsoft.com> * Workflow and playwright test updates (#1360) Signed-off-by: ytimocin <ytimocin@microsoft.com> Signed-off-by: sk593 <shruthikumar@microsoft.com> * Changing the place of page.reload in the tests (#1492) Signed-off-by: ytimocin <ytimocin@microsoft.com> Signed-off-by: sk593 <shruthikumar@microsoft.com> * Adding a job that will purge AWS EKS clusters every 6 hours (#1462) Signed-off-by: ytimocin <ytimocin@microsoft.com> Signed-off-by: sk593 <shruthikumar@microsoft.com> * Update all deps of all apps (#1536) Signed-off-by: ytimocin <ytimocin@microsoft.com> Signed-off-by: sk593 <shruthikumar@microsoft.com> * Updating the runner of purge azure test resources workflow (#1537) Signed-off-by: ytimocin <ytimocin@microsoft.com> Signed-off-by: sk593 <shruthikumar@microsoft.com> * Update purget test resources to fix the workflow failures (#1552) Signed-off-by: ytimocin <ytimocin@microsoft.com> Signed-off-by: sk593 <shruthikumar@microsoft.com> * Fixing the eShop catalog load issue for the UI tests (#1548) Signed-off-by: ytimocin <ytimocin@microsoft.com> Signed-off-by: sk593 <shruthikumar@microsoft.com> --------- Signed-off-by: Reshma Abdul Rahim <reshmarahim.abdul@microsoft.com> Signed-off-by: ytimocin <ytimocin@microsoft.com> Signed-off-by: Will Tsai <28876888+willtsai@users.noreply.github.com> Signed-off-by: Will <28876888+willtsai@users.noreply.github.com> Signed-off-by: sk593 <shruthikumar@microsoft.com> Co-authored-by: Will <28876888+willtsai@users.noreply.github.com> Co-authored-by: Reshma Abdul Rahim <61033581+Reshrahim@users.noreply.github.com> Co-authored-by: Yetkin Timocin <ytimocin@microsoft.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.