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 <- production-staging #353

Merged
merged 73 commits into from
Oct 11, 2021
Merged

production <- production-staging #353

merged 73 commits into from
Oct 11, 2021

Conversation

arnab-p
Copy link
Contributor

@arnab-p arnab-p commented Oct 8, 2021 •
edited by SumanthPuram
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

Fix #1

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

sayan-mitra and others added 30 commits March 24, 2021 12:22
@sayan-mitra
add dist files
e03d983
@akashrpo
Snap Pixel Initial Commit
3a3a9fb
@akashrpo
add cookie data check and reformat code
b919b30 
Signed-off-by: Akash Gupta <[email protected]>
@akashrpo
add cases for empty payload
1107c1d 
Signed-off-by: Akash Gupta <[email protected]>
@akashrpo
modified payload mapping
56023e9 
Signed-off-by: Akash Gupta <[email protected]>
@akashrpo
add edge cases and refactor code
df1ab5a 
Signed-off-by: Akash Gupta <[email protected]>
@akashrpo
trim the event value
20c12a8 
Signed-off-by: Akash Gupta <[email protected]>
@akashrpo
add log and change event name
b41e1d1 
Signed-off-by: Akash Gupta <[email protected]>
@arajguha
minor update
703eacb
@akashrpo
remove page_view event from the list
da645d2 
Signed-off-by: Akash Gupta <[email protected]>
@arajguha
Merge pull request #310 from rudderlabs/snap_pixel
234e159 
Snap Pixel Integration
@akashrpo
update custom event list
5dc5d33
@arajguha
Merge pull request #313 from rudderlabs/snap_pixel
71925b8 
Snap pixel update
@arajguha
refactor event names in object
987e747
@arajguha
refactor event names
2075cdd
@arajguha
Merge pull request #317 from rudderlabs/snap_pixel
cd860cd 
refactor event names in object
@akashrpo
ProfitWell Initial Commit
1ef638f
@akashrpo
refactor code
5536ca0
@akashrpo
refactor code
28a1882
@arajguha
Merge pull request #323 from rudderlabs/production-profitwell
d505183 
Profitwell Integration
@akashrpo
remove cookieData
008fe6d
@akashrpo
remove throw error message
8639a94
@arajguha
Merge pull request #326 from rudderlabs/production-profitwell
4dfbecd 
Profitwell device-mode update
@akashrpo
correct payload key
37c7ae0
@arajguha
Merge pull request #327 from rudderlabs/production-profitwell
8a8603e 
Profitwell device-mode - Issue
@arajguha
update profitwell identify
3d39d41
@arajguha
remove unused import
140eefa
@arajguha
Delete test-tool.yaml
87c8d5a
@arajguha
Merge pull request #329 from rudderlabs/production-profitwell
d11f7e0 
update profitwell identify
@arajguha
move marketing tracking to identify
75760b4
shrouti1507 and others added 27 commits September 21, 2021 11:38
@shrouti1507
edit in traits and object to array function
8decc8a
@arajguha
minor update
91e5d72
@arajguha
Merge pull request #335 from rudderlabs/sentry_integration
fcb3879 
Sentry integration
@shrouti1507
edit in config
6fbc04a
@shrouti1507
init code transferred to isReady for loading issue
3881888
@shrouti1507
code refactored and ready is modified
be67b8a
@arajguha
minor update
bb3d3ef
@arajguha
Merge pull request #337 from rudderlabs/sentry_integration
e600d4d 
Sentry Integrations
@arajguha
update identify
f4a185c
@arajguha
Merge pull request #338 from rudderlabs/sentry_integration
89e169b 
update identify
@arajguha
update scriptloader id
87f2725
@arajguha
Merge pull request #339 from rudderlabs/sentry_integration
3f437fe 
update scriptloader id
@shrouti1507
edit in rewriteFrames for ForEach
fbbf8bd
@arajguha
Merge pull request #343 from rudderlabs/fix/sentry-staging
914007f 
edit in rewriteFrames for ForEach
@saikumarrs
Enhancement: Updated dist for npm
7ae1fd3 
- .Updated gitignore to exclude npm package dir
- npm version updated to 1.0.19
- README.md updated to latest
@saikumarrs
Merge branch 'production-npm-staging' into production-staging
e1fccfc
@dalyd14
resolved conflicts, added Quatum metric integration
dde8141
@dalyd14
Merge branch 'production-staging' into QuatumMetric-int
b923ac1
@dalyd14
changed config setting to siteID
644cf3e
@ruchiramoitra
QuatumMetric-int:Addressed Review comments
c11a185
@ruchiramoitra
QuatumMetric-int:Removed siteID
0a47eb5
@arnab-p
Merge pull request #346 from rudderlabs/QuatumMetric-int
ad4aea8 
Quantum metric int
@dalyd14
Feature: Allow Pinterest top-level property mapping i.e. revenue -> v…
6be4e59 
…alue (#322)

* removed checkout_id support and replaced it with order_id to align better with ecomm spec

* added feature to allow for toplevel property mapping
@dalyd14
removed bug that strips property key value pairs
f625307
@arnab-p
Merge pull request #352 from rudderlabs/GA4-stripping
e9000d8
@saikumarrs
Merge branch 'production' into production-staging
e033343
@saikumarrs
Feature: Configure storage type from options (#336)
3de3fde 
* Initial cut to configure storage type from options

* Update option parameter name

* Code-refactored and minor bug fixed
@saikumarrs saikumarrs self-requested a review October 11, 2021 12:39
@saikumarrs saikumarrs merged commit bbdb5c9 into production Oct 11, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@saikumarrs saikumarrs saikumarrs 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.
9 participants
@arnab-p @saikumarrs @sayan-mitra @akashrpo @arajguha @utsabc @shrouti1507 @dalyd14 @ruchiramoitra