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

DS-811 Wrapper Element #2522

Merged
merged 11 commits into from Aug 10, 2022
Merged

DS-811 Wrapper Element #2522

merged 11 commits into from Aug 10, 2022

Conversation

MarcinMr
Copy link
Collaborator

@MarcinMr MarcinMr commented Aug 2, 2022

Jira

https://pegadigitalit.atlassian.net/browse/DS-811

Summary

A new wrapper element is created.

Details

A new wrapper is a basic element that contains a basic schema with content and attributes.

How to test

Pull the branch. Check the wrapper docs and confirm it's a good equivalent of the old wrapper object.

Release notes

The o-bolt-wrapper object is deprecated. Please use the e-bolt-wrapper element instead.

@github-actions github-actions bot added the type: feature List this PR in the 'Features' section of the release notes. label Aug 2, 2022
@colbytcook colbytcook temporarily deployed to feature/DS-811-Wrapper-Element--branch-preview August 2, 2022 10:33 Inactive
@colbytcook colbytcook requested a deployment to feature/DS-811-Wrapper-Element--3dbc663--commit-preview August 2, 2022 10:54 In progress
@colbytcook colbytcook temporarily deployed to feature/DS-811-Wrapper-Element--branch-preview August 2, 2022 11:09 Inactive
Copy link
Contributor

@colbytcook colbytcook left a comment

Choose a reason for hiding this comment

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

Approved

@colbytcook colbytcook requested a deployment to feature/DS-811-Wrapper-Element--8bd57ea--commit-preview August 4, 2022 16:07 Abandoned
@colbytcook colbytcook requested a deployment to feature/DS-811-Wrapper-Element--branch-preview August 4, 2022 16:14 Abandoned
Copy link
Collaborator

@danielamorse danielamorse left a comment

Choose a reason for hiding this comment

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

@MarcinMr I left some copy suggestions. If this sounds good, please apply the changes and commit (you can do this right in Github FYI).

Otherwise, this looks great! I created a small test page locally to verify it works just like o-bolt-wrapper.

@colbytcook colbytcook temporarily deployed to feature/DS-811-Wrapper-Element--branch-preview August 10, 2022 10:56 Inactive
MarcinMr and others added 2 commits August 10, 2022 13:12
Co-authored-by: Dan Morse <danielamorse@gmail.com>
Co-authored-by: Dan Morse <danielamorse@gmail.com>
Co-authored-by: Dan Morse <danielamorse@gmail.com>
@MarcinMr
Copy link
Collaborator Author

@danielamorse approved your suggestions

@danielamorse danielamorse merged commit e8b9810 into master Aug 10, 2022
@danielamorse danielamorse deleted the feature/DS-811-Wrapper-Element branch August 10, 2022 14:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: feature List this PR in the 'Features' section of the release notes.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants