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.

Sign up for GitHub

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

Jump to bottom

LPS-77329 Reorder resulution position in the plugins exports declaration #61344

Closed
wants to merge 17 commits into from
Closed

LPS-77329 Reorder resulution position in the plugins exports declaration #61344

wants to merge 17 commits into from

Conversation

clovisdasilvaneto
Copy link

No description provided.

Bruno Basto and others added 17 commits July 9, 2018 18:01
LPS-83303 We also need to send the document title in the analytics ev…
008f6d2 
…ent payload
@clovisdasilvaneto
LPS-77329 Generated package.lock
057d814
@clovisdasilvaneto
LPS-77329 Add navigation between test pages to make testing manual co…
faeca23 
…ntexts easier
@clovisdasilvaneto
LPS-77329 Save the actual user context in the local storage to send t…
1123b09 
…he events with the right context
@clovisdasilvaneto
LPS-77329 Add context hash in the events object to compare it when th…
a78c97b 
…e backend request is made
@clovisdasilvaneto
LPS-77329 declare instance.contexts to use this in the analytics requ…
388b8a8 
…ests
@clovisdasilvaneto
LPS-77329 Import meta middleware for creating the context if it doesn…
33b5d27 
…'t exists
@clovisdasilvaneto
LPS-77329 Import object-hash to unhash the context before sent the an…
e9c6457 
…alytics requests to the backend
@clovisdasilvaneto
LPS-77329 Divider analytics requests by context to sent each event wi…
5910791 
…th they own context
@clovisdasilvaneto
LPS-77329 Parse the actual context for the getRequestBody to use it t…
d50087b 
…o get the events that are part of this informed context
@clovisdasilvaneto
LPS-77329 Remove inline events conditions
8c1afe9
@clovisdasilvaneto
LPS-77329 Get each event from their respective context by a condition…
af55530 
… that compares the applicationId, eventId and the serializedContext of the current event and parse them for the actual context for the requestBody variable
@clovisdasilvaneto
LPS-77329 Add condition to avoid duplicating the contexts in local st…
76cd643 
…orage
@clovisdasilvaneto
LPS-77329 Remove each context that doesn't have any events attached t…
d292662 
…o it from the local storage
LPS-77329 SF
74040a0
@clovisdasilvaneto
LPS-77329 Ignore some props in the objectHash method configuration
ee0322e
@clovisdasilvaneto
LPS-77329 Reorder resulution position in the plugins exports declaration
c7b06e4
@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.

@clovisdasilvaneto clovisdasilvaneto mentioned this pull request Jul 13, 2018
Closed
@liferay-continuous-integration
Copy link
Collaborator

✔️ ci:test:sf - 1 out of 1 jobs passed in 1 minute 32 seconds 57 ms

Click here for more details.

Base Branch:

Branch Name: master
Branch GIT ID: 3a855dcce4ac1b504980669fdcbc357cfbf3511d

Sender Branch:

Branch Name: pr-56
Branch GIT ID: c7b06e4e87fa31abc14e97628d8b34994309cdff

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

@liferay-continuous-integration
Copy link
Collaborator

✔️ ci:test:relevant - 20 out of 20 jobs passed in 54 minutes 52 seconds 77 ms

Click here for more details.

Base Branch:

Branch Name: master
Branch GIT ID: 3a855dcce4ac1b504980669fdcbc357cfbf3511d

Copied in Private Modules Branch:

Branch Name: master-private
Branch GIT ID: ee4dc87111ad278cdf1b10f685ce69d69ccdf2ab

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

@clovisdasilvaneto
Copy link
Author

clovisdasilvaneto commented Jul 13, 2018 •
edited

cc @marcellustavares

@brianchandotcom
Copy link
Owner

Merged. Thank you.
View total diff: 361c264...bc86e3a

@brianchandotcom
Copy link
Owner

@clovisdasilvaneto @brunobasto see bc86e3a thx.

@clovisdasilvaneto clovisdasilvaneto deleted the pr-56 branch July 13, 2018 21:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
ci:test:relevant - success ci:test:sf - success
Projects
None yet
Milestone
No milestone
3 participants
@clovisdasilvaneto @liferay-continuous-integration @brianchandotcom