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

Production -> master #183

Merged
merged 127 commits into from
Jan 15, 2021
Merged

Production -> master #183

merged 127 commits into from
Jan 15, 2021

Conversation

sayan-mitra
Copy link
Contributor

@sayan-mitra sayan-mitra commented Jan 4, 2021 •
edited
Loading

Description of the change

Description here

Type of change

  • Bug fix (non-breaking change that fixes an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

Related issues

Checklists

Development

  • Lint rules pass locally
  • The code changed/added as part of this pull request has been covered with tests
  • All tests related to the changed code pass in development

Code review

  • This pull request has a descriptive title and information useful to a reviewer. There may be a screenshot or screencast attached
  • "Ready for review" label attached to the PR and reviewers mentioned in a comment
  • Changes have been reviewed by at least one other engineer
  • Issue from task tracker has a link to this pull request

This change is Reviewable

arnab-p and others added 30 commits August 12, 2020 13:54
@arnab-p
Merge pull request #118 from rudderlabs/master
5f5d1eb
@sayan-mitra
update npm module
62020d6
@prabrishac
fix for page properties
6912a14
@sayan-mitra
init changes
b6cacb7
@sayan-mitra
add polyfill and other changes
c1b5401
@sayan-mitra
add node modules transpilation
d2a505b
@sayan-mitra
change tostring for ie
ce0f45e
@sayan-mitra
change tostring for ie
406437a
@sayan-mitra
Merge pull request #122 from rudderlabs/fix/page_properties
10c68ae 
Fix: page properties
@sayan-mitra
add query parse logic from url and send events
1faabd5
@sayan-mitra
bump version
fe16ef5
@sayan-mitra
Merge pull request #128 from rudderlabs/query_string_parse_production
5c63766 
add query parse logic from url and send events ---> Production
@ruchiramoitra @arnab-p
added tvsquared as destination
3a85480
@ruchiramoitra @arnab-p
minor changes from review comment and formatting
214727b
@ruchiramoitra @arnab-p
changed return avlue for isloaded and isready
0b38d08
@ruchiramoitra @arnab-p
minor changes
e03e816
@prabrishac @arnab-p
bug fix
6b66558
@arnab-p
Bumped version
668baa0
@arnab-p
Merge pull request #134 from rudderlabs/tvsquared-production
9503d6a 
TVSquared destination
@sayan-mitra
copy changes from dev
9ef61ec
@sayan-mitra
bump version
f0435b6
@arnab-p
Merge pull request #139 from rudderlabs/gaNameTrackerProd
b64e491 
Ga name tracker ---> Production
@arnab-p
Forced hubspot to load on https
cd85110
@arnab-p
Merge pull request #149 from rudderlabs/hubspot-https-fix
9777852
@dhawal1248 @sayan-mitra
make event retry count configurable
66e9648
@sayan-mitra
add support for queueOptions
64cd4cd
@sayan-mitra
send campaign parsed info
fa07730
@sayan-mitra
add deps
709bde0
@sayan-mitra
bump version
13f4311
@sayan-mitra
Merge pull request #148 from rudderlabs/combined_branch_2
b1852d7 
merge campaign and configurable queue
desusai7 and others added 25 commits December 23, 2020 12:43
@desusai7
Added Appcues
0cdea95
@desusai7
Implemented Identify,Track,Page
11469b1
@desusai7
Compiled browser.js to tests/html
8c45a1a
@desusai7
Implemented reset, handling Appcues default calls
8efcb7c
@desusai7
Added Feature to send Appcues flow events to all destinations
907f075
@desusai7
Modified Testcases in tests/html/script-test.html
69f0152
@desusai7
Made Minor Fixes and Added Appcues to integration_cname and client_se…
14f43be 
…rver_name.js
@desusai7
Pushing Updated browser.js
b0c1f11
@Shivam4415
GA4 Bug fixes
f7b904d
@sayan-mitra
Merge pull request #177 from rudderlabs/ga4/bug-fixes
57616b5 
GA4 Bug fixes
@Shivam4415
GA4 Bug fixes- added logic of sendUserId in init function when script…
d05e725 
…s is loading
@sayan-mitra
Merge pull request #178 from rudderlabs/ga4/bug-fixes
d58262c 
GA4 Bug fixes- added logic of sendUserId in init function when script…
@Shivam4415
GA4 Bug fixes- flattening track call json payload
1df4fc6
@sayan-mitra
Merge pull request #179 from rudderlabs/ga4/bug-fixes
cf2ab5b 
GA4 Bug fixes- flattening track call json payload
@Shivam4415
GA4 Bug fixes- added logic to not send page view event when blockPage…
ead5007 
…View is false
@sayan-mitra
merge production
68fbd77
@sayan-mitra
resolve conflict and bump version
dee0170
@sayan-mitra
format
d864b29
@sayan-mitra
Merge pull request #180 from rudderlabs/ga4/bug-fixes
2059243 
GA4 Bug fixes- page view will not trigger with identify call when blockPageView flag is true.
@prabrishac
Merge pull request #174 from rudderlabs/production-staging
93586be 
Production <--  Staging
@prabrishac
Merge branch 'production-staging' into appCuesDevelop
4f4930c
@prabrishac
Merge pull request #176 from rudderlabs/appCuesDevelop
5355040 
Appcues Web SDK
@sayan-mitra
Merge pull request #182 from rudderlabs/production-staging
eacba51 
Production <- staging
@sayan-mitra
merge master
83170c7
@sayan-mitra
update dist files
d2bcd18
@sayan-mitra sayan-mitra requested a review from prabrishac January 4, 2021 14:52
@sayan-mitra sayan-mitra changed the title Production merge Production -> master Jan 4, 2021
github-actions[bot]
github-actions bot reviewed Jan 4, 2021
View reviewed changes
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

size-limit report

Path Size Loading time (3g) Running time (snapdragon) Total time
dist/rudder-analytics.min.js 63.1 KB (+9.38% 🔺) 1.3 s (+9.38% 🔺) 365 ms (+10.93% 🔺) 1.7 s

@sayan-mitra sayan-mitra merged commit f2ae2b7 into master Jan 15, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@github-actions github-actions[bot] github-actions[bot] left review comments

@prabrishac prabrishac prabrishac approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
7 participants
@sayan-mitra @prabrishac @arnab-p @ruchiramoitra @dhawal1248 @Shivam4415 @desusai7