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

Merge changes to staging for 4.7.2 #6410

Merged
merged 60 commits into from
May 6, 2024
Merged

Merge changes to staging for 4.7.2 #6410

merged 60 commits into from
May 6, 2024

Conversation

ramyaragupathy
Copy link
Member

@ramyaragupathy ramyaragupathy commented May 6, 2024

What type of PR is this? (check all applicable)

  • ❓ Other 4.7.2 release

tsmock and others added 30 commits April 16, 2024 08:16
Signed-off-by: Taylor Smock <[email protected]>
Signed-off-by: Taylor Smock <[email protected]>
* timers (such as advanceTimersByTime)
* clicks

Signed-off-by: Taylor Smock <[email protected]>
Signed-off-by: Taylor Smock <[email protected]>
Signed-off-by: Taylor Smock <[email protected]>
Signed-off-by: Taylor Smock <[email protected]>
quickstart.js has the same spacing as it had previously (which comes
from the empty <p/>).

Signed-off-by: Taylor Smock <[email protected]>
* `Sign Up` button
* comment boxes
* actionsForm.js (mostly buttons)

Signed-off-by: Taylor Smock <[email protected]>
This reduces the main chunk size from 3.65 MB to 2.56 MB.
Something about the commit "Use CRACO to workaround CRA #11770" caused
several large components to be added to the main.*.js bundle.

All suspense elements are *roughly* the same size as the expected text
editor box, plus or minus a row.

Signed-off-by: Taylor Smock <[email protected]>
* Avoid default imports in ReactJS
  There were quite a few files where we imported react, but didn't
  actually use it.
* Use non-default imports where possible
* Use named imports where possible

Signed-off-by: Taylor Smock <[email protected]>
Signed-off-by: Taylor Smock <[email protected]>
Signed-off-by: Taylor Smock <[email protected]>
Signed-off-by: Taylor Smock <[email protected]>
This should fix some timeout issues.
@github-actions github-actions bot added dependencies Pull requests that update a dependency file codebase: frontend labels May 6, 2024
Copy link

sonarcloud bot commented May 6, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@ramyaragupathy ramyaragupathy merged commit 252d9e0 into main May 6, 2024
10 of 14 checks passed
@ramyaragupathy ramyaragupathy added this to the v4.7.2 milestone May 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file scope: frontend scope: infrastructure
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants