Skip to content

Conversation

@AnthonyLedesma
Copy link
Member

@AnthonyLedesma AnthonyLedesma commented Nov 1, 2023

Description

Moved Animation Styles. New Tests. New Workflow to test only on changed files. Bump to WP-next version.

Screenshots

animationOnCoreBlocks

Types of changes

Splitting styles apart into new Webpack compile.
Importing new styles always.
New tests to check for the condition of always-loaded styles.
New workflow for testing velocity.

How has this been tested?

Tested manually as well as new tests in CICD

Acceptance criteria

Should allow for animation to properly function on the Editor and Front end for all blocks, CoBlocks and Core.

Checklist:

  • My code is tested
  • My code follows accessibility standards
  • My code has proper inline documentation
  • I've included any necessary tests
  • I've included developer documentation
  • I've added proper labels to this pull request

@AnthonyLedesma AnthonyLedesma self-assigned this Nov 3, 2023
@AnthonyLedesma AnthonyLedesma added the [Type] Bug Something that is not working as expected label Nov 3, 2023
Copy link
Member

@mtashjianjr-godaddy mtashjianjr-godaddy left a comment

Choose a reason for hiding this comment

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

Overall this looks great! Really great job putting some efficiency into our pipeline by only testing what changed!

@EvanHerman
Copy link
Contributor

EvanHerman commented Nov 6, 2023

PR changes are working perfectly on my end, great work on that! Just some minor changes above and I think it's good to go.

@EvanHerman EvanHerman self-requested a review November 6, 2023 17:51
@AnthonyLedesma AnthonyLedesma dismissed mtashjianjr-godaddy’s stale review November 6, 2023 18:12

Attempted optimize but reverted as good enough for now.

@AnthonyLedesma AnthonyLedesma merged commit f67902e into master Nov 6, 2023
@AnthonyLedesma AnthonyLedesma deleted the enqueue_animation_css_always branch November 6, 2023 18:23
@AnthonyLedesma AnthonyLedesma added this to the Next Release milestone Nov 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

[Type] Bug Something that is not working as expected

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants