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

LPS-114004 Replace layout classes with new clay taglibs in account-admin-web #89245

Conversation

liferay-continuous-integration
Copy link
Collaborator

Forwarded from: drewbrokke#1076 (Took 1 ci:forward attempt in 3 minutes)

@eduardoallegrini
@drewbrokke

Original pull request comment:
Hey @drewbrokke , this is a simple code replacement that uses the new Clay Taglibs for old code like <div class="sheet sheet-lg"> > <clay:sheet>.

Please, could you check if the module is still showing correctly or reply with instructions to see it?

I can't provide an official documentation right now, we have discussed the whole thing in this pr (liferay/clay#3204) but I'm creating a quick list of the usages found so far in this google sheet (https://docs.google.com/spreadsheets/d/1mXjvokQRqSMuQtpghUAFLb2Igd21zgJvc8whUmV_-EU/edit?usp=sharing)

/cc @jbalsas @marcoscv-work

✔️ ci:test:sf - 1 out of 1 jobs passed in 4 minutes

Click here for more details.

Base Branch:

Branch Name: master
Branch GIT ID: a0bfd3552193bb33687ef3677fe4879447e04d6a

Sender Branch:

Branch Name: LPS-114004_layouts_account-admin-web
Branch GIT ID: 15d34fe77dc6f935e615e1f220a1ca4e821807b2

1 out of 1jobs PASSED
1 Successful Jobs:
For more details click here.

✔️ ci:test:stable - 19 out of 19 jobs passed

✔️ ci:test:relevant - 42 out of 42 jobs passed in 1 hour 12 minutes

Click here for more details.

Base Branch:

Branch Name: master
Branch GIT ID: 884cd741c11f60df048c695bd2567d3c69bd411e

Copied in Private Modules Branch:

Branch Name: master-private
Branch GIT ID: 6845814bf7a2b1b38f7c5a057c77bb287029faf4

ci:test:stable - 19 out of 19 jobs PASSED
19 Successful Jobs:
ci:test:relevant - 42 out of 42 jobs PASSED
42 Successful Jobs:
For more details click here.

@liferay-continuous-integration
Copy link
Collaborator Author

To conserve resources, the PR Tester does not automatically run for forwarded pull requests.

@drewbrokke
Copy link

@brianchandotcom
Copy link
Owner

Merged. Thank you.
View total diff: 7422792...382a403

@liferay-continuous-integration liferay-continuous-integration deleted the ci-forward-LPS-114004_layouts_account-admin-web-pr-1076-sender-eduardoallegrini-ts-1590531470309 branch June 3, 2020 10:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants