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-106021 Assert image src is correct when manipulating the parent #83101

Closed
wants to merge 22 commits into from

Conversation

victorg1991
Copy link

carolmariaabb and others added 22 commits January 10, 2020 14:11
**Attention to mocked data, since there aren't any clarification if we gonna still using `getAssetMappingFieldsURL` and `mappingFieldsURL`
Here we are adding a raw preview of the selected value, when
the processor calls changedCallback. This is not definitive, as
we need to deal with the backend service and the React render
process, but still allow us to temporary test processors.
@liferay-continuous-integration
Copy link
Collaborator

CI is automatically triggering "ci:test:sf" and "ci:test:relevant" for this pull to run Source Formatter and relevant tests.

Comment "ci:test" to run the full PR Tester for this pull.

@victorg1991
Copy link
Author

hey @brianchandotcom I needed some commit in your master that are not in Liferay repo for this pr. So I've tried to rebase with your master and removed all commits that mentioned @jpince, let's see if this works :)

@liferay-continuous-integration
Copy link
Collaborator

✔️ ci:test:sf - 1 out of 1 jobs passed in 14 minutes 9 seconds 101 ms

Click here for more details.

Base Branch:

Branch Name: master
Branch GIT ID: 4a20027f18eab65ddd0c786fd858e7b2a5b24f71

Sender Branch:

Branch Name: pr-139
Branch GIT ID: ffb41155619f915b9d7c67f8ad9739a063bf9c3d

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

@liferay-continuous-integration
Copy link
Collaborator

❌ ci:test:relevant - 0 out of 1 jobs passed in 10 minutes 56 seconds 444 ms

Click here for more details.

Base Branch:

Branch Name: master
Branch GIT ID: 1991cde6e2f3ef10689a7b32c819cd91cf9faa55

ci:test:relevant - 0 out of 1 jobs PASSED
For more details click here.

Failures unique to this pull:

  1. test-portal-acceptance-pullrequest(master)
    Job Results:

    0 Jobs Passed.
    1 Job Failed.

    A CI failure has occurred. QA Engineering has been notified. Please try again later or contact QA Engineering for an update.

For upstream results, click here.

@jpince
Copy link
Collaborator

jpince commented Jan 10, 2020

@victorg1991 rebase error

@victorg1991
Copy link
Author

gonna try again @jpince

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.

8 participants