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

[Major release] Angular wrappers and React name changed #2045

Open
wants to merge 8 commits into
base: alpha
Choose a base branch
from

Conversation

vanessatran-ddi
Copy link
Collaborator

I created this drafted pull request to compare/update with alpha branch.

@vanessatran-ddi vanessatran-ddi changed the title [Do not test] add new angular components [Major release] Angular wrappers and React name changed Aug 28, 2024
@vanessatran-ddi vanessatran-ddi force-pushed the angular-components branch 3 times, most recently from 94b91fd to bd9c6e0 Compare September 9, 2024 18:49
@vanessatran-ddi vanessatran-ddi marked this pull request as ready for review September 9, 2024 18:49
Copy link
Collaborator

@chrisolsen chrisolsen left a comment

Choose a reason for hiding this comment

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

Can you shorten your breaking change commit messages using the following format

fix(#999): [some commit message]

BREAKING CHANGE: [what is causing the breaking change]

.github/workflows/pull-request.yml Show resolved Hide resolved
@vanessatran-ddi vanessatran-ddi force-pushed the angular-components branch 4 times, most recently from 4c9c608 to 4527899 Compare September 16, 2024 19:31
package.json Outdated Show resolved Hide resolved
@vanessatran-ddi vanessatran-ddi force-pushed the angular-components branch 2 times, most recently from 616dfcf to e32cdd4 Compare September 17, 2024 19:14
@chrisolsen chrisolsen self-requested a review September 17, 2024 20:15
chrisolsen
chrisolsen previously approved these changes Sep 17, 2024
@vanessatran-ddi vanessatran-ddi force-pushed the angular-components branch 2 times, most recently from edeee57 to 766feaf Compare November 20, 2024 19:54
@vanessatran-ddi vanessatran-ddi force-pushed the angular-components branch 3 times, most recently from 2398b31 to 05eaea2 Compare November 29, 2024 22:29
@vanessatran-ddi vanessatran-ddi self-assigned this Nov 29, 2024
Comment on lines 187 to 189
<slot>
{text}
</slot>
Copy link
Collaborator

Choose a reason for hiding this comment

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

This could be a breaking change

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Yes I put it back

<div class="text" data-testid="text">
      <slot></slot>
        {text}
    </div>

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.

2 participants