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

SD-107645 | Products ingesting incorrectly #21

Open
wants to merge 456 commits into
base: main
Choose a base branch
from

Conversation

emreisik95
Copy link

A summary of your pull request, including the what change you're making and why.

Testing

Include any additional information about the testing you have completed to
ensure your changes behave as expected. For a speedy review, please check
any of the tasks you completed below during your testing.

  • Added unit tests for new functionality
  • Tested end-to-end using the local server
  • [Segmenters] Tested in the staging environment

joe-ayoub-segment and others added 30 commits March 18, 2024 18:56
* [Snap V3 CAPI] use pixel id for offline events

* fix test

* Add default value for item_category

---------

Co-authored-by: David Bordoley <[email protected]>
segmentio#1926)

* Moves ad account and campaign fields from regular field section to hook field section

* When creating or updating or getting conversion rule, pull from hookInputs.adAccountId rather than the payload

* Associate conversion as a part of the conversion creation or update rather than as a part of streaming an event

* Updates api unit tests

* Removes campaign conversion related unit test from streamConversion tests

* removes stray tab from linkedin description

* Updates snapshots

* Removes timestamps from snapshot since they are flaky

* core: Allows hook input fields to be arrays of values

* Moves campaign association to after the conversion rule is created or updated

* Uses a setter method to set the conversion rule id in the linkedin client rather than setting it in the constructor. This is cleaner since we need the client to create the ID and so don't know what that ID will be yet until it is created

* Allows hookInputs to be passed in the data object to the dynamic field executor

* Removes tuple return from parseIdFromUrn(), error will be returned if that method returnd undefined instead
segmentio#1936)

* external_id is only shown if role is not lead

* Adds dependsOn to new LinkedIn fields

* Revert "external_id is only shown if role is not lead"

This reverts commit 5468155.

* Google Enhanced Conversions: Only show restatement_value/currency_code if attribution type is not RETRACTION
* Add mantle action destination

* Add defaults and description for mantle destination
…egmentio#1934)

* fix: partially formatted E164 numbers failing to send

* chore: change test numbers

* chore: added more tests around external id
* update to track event to use event capture

* update to identify call and fields

* update to identify test

* update to test suite
* timeout error 429 and stringify

* Updated stringify setting

* fixed recipient spelling

* refactor

* removed console log statement

* adding stats context

* removing bad validation

---------

Co-authored-by: Joe Ayoub <[email protected]>
Registering new Integrations
* Try new release tag

* fix npm package format

* Rename steps

* use context params

* Refactor and simplify code

* minor fixes and cleanup

* count publish commits only for release

* refactor

* cosmetic changes

* exclude scripts from tsconfig and eslint
* Adds an else statement to fix an issue were conversion rules were always created and never updated

* Removes the userIds array field in favor of separate individual fields which an array will be constructed out of. Also hashes user email before sending

* Removes manually thrown error when first or last name are not provided for userInfo object, since AJV throws that error for us

* updates streamConversion unit tests

* Adds another unit test, it's not passing yet but it will, one day

* Updates snapshot tests and snapshots

* Deletes snapshot unit tests that were just a copy of existing snapshot unit tests, and therefore redundant

* Fixes expected nock body

* Passes userIds array correctly, with idType and idValue params rather than type and value

* Updates unit tests to use idType and idValue instead
* Bootstrap sendTrack action

* Add timestamp

* Add datadog stat

* Add description

* add missing engage_space property

* minor fixes
* updates to Kafka

* updates to kafka

* minor change

* updating tests

* removing bad reference

* removing partitioner setting field
* Provides some simple defaults for streamConversion. Adds unit tests. Adds to defaults section on README

* Hardcodes a messageID in test event and tests default mappings with it

* Fixes broken hyperengage unit test which broke for some unknown reason???
varadarajan-tw and others added 9 commits July 2, 2024 19:52
* Publish

 - @segment/[email protected]
 - @segment/[email protected]
 - @segment/[email protected]
 - @segment/[email protected]
 - @segment/[email protected]
 - @segment/[email protected]
 - @segment/[email protected]
 - @segment/[email protected]
 - @segment/[email protected]
 - @segment/[email protected]
 - @segment/[email protected]
 - @segment/[email protected]
 - @segment/[email protected]
 - @segment/[email protected]
 - @segment/[email protected]
 - @segment/[email protected]
 - @segment/[email protected]
 - @segment/[email protected]
 - @segment/[email protected]
 - @segment/[email protected]
 - @segment/[email protected]
 - @segment/[email protected]
 - @segment/[email protected]
 - @segment/[email protected]
 - @segment/[email protected]
 - @segment/[email protected]
 - @segment/[email protected]
 - @segment/[email protected]
 - @segment/[email protected]
 - @segment/[email protected]
 - @segment/[email protected]
 - @segment/[email protected]
 - @segment/[email protected]
 - @segment/[email protected]
 - @segment/[email protected]
 - @segment/[email protected]
 - @segment/[email protected]
 - @segment/[email protected]
 - @segment/[email protected]
 - @segment/[email protected]
 - @segment/[email protected]
 - @segment/[email protected]
 - @segment/[email protected]

* Use from-package workflow instead of from-git

* Comment out the release script
* adding logging

* removing logging

* refactor

* making some fields visible
* empty commit

* Publish

 - @segment/[email protected]
 - @segment/[email protected]
 - @segment/[email protected]
 - @segment/[email protected]
 - @segment/[email protected]
 - @segment/[email protected]
 - @segment/[email protected]
 - @segment/[email protected]
 - @segment/[email protected]
 - @segment/[email protected]
 - @segment/[email protected]
 - @segment/[email protected]
 - @segment/[email protected]
 - @segment/[email protected]
 - @segment/[email protected]
 - @segment/[email protected]
 - @segment/[email protected]
 - @segment/[email protected]
 - @segment/[email protected]
 - @segment/[email protected]
 - @segment/[email protected]
 - @segment/[email protected]
 - @segment/[email protected]
 - @segment/[email protected]
 - @segment/[email protected]
 - @segment/[email protected]
 - @segment/[email protected]
 - @segment/[email protected]
 - @segment/[email protected]
 - @segment/[email protected]
 - @segment/[email protected]
 - @segment/[email protected]
 - @segment/[email protected]
 - @segment/[email protected]
 - @segment/[email protected]
 - @segment/[email protected]
 - @segment/[email protected]
 - @segment/[email protected]
 - @segment/[email protected]
 - @segment/[email protected]
 - @segment/[email protected]
 - @segment/[email protected]
 - @segment/[email protected]
 - @segment/[email protected]
 - @segment/[email protected]
* adding arrayeditor option

* update description
@emreisik95 emreisik95 requested review from ryigit and rbcet July 8, 2024 09:35
@emreisik95 emreisik95 self-assigned this Jul 8, 2024
@emreisik95 emreisik95 changed the base branch from main to feature/SD-98474 July 8, 2024 09:36
* Introduce new release flow

* quick check to validate push permissios

* revert changes

* refactor release.sh

* revert release.sh

* Revert unwanted changes

* Restore gh pat token change
@emreisik95 emreisik95 changed the base branch from feature/SD-98474 to main July 8, 2024 13:46
AnkitSegment and others added 13 commits July 9, 2024 12:20
…egmentio#2081)

* added depends_on conditions on salesforce actions

* updated conditions for depends_on fields

* Updated depends on condition for delete operation in salesforce actions

* Updated depends on conditions for salesforce actions

Contact, lead, opportunity

* Added depends on for cases action in salesforce
We needed to send a few other fields from the context to make our app
more robust.
* Add platform name

* Set platformName optional, and use platformId as a default value
* updated the mapping

* update delete object

* added comments

* fixed the mapping
* initial commit

* fix failing tests

* updated saveUser action

* updated saveOrder action

* updated saveEvent action

* added action descriptions

* updated presets

* flattened saveEvent mapping

* fixed saveEvent.perform

* fixed saveEvent mapping

* updated snapshots

* updated snapshots

* fixed saveEvent payload structure

* don't include empty objects in payload

* updated

* progress

* restructure events

* remove defaults for optional fields

* restore address field

* update untranslated title default values

* updated snapshots

* remove unused import

* fixed default paths

* reorganized saveUser fields

* updated saveBaseEvent fields

* fixed default_address default in saveUser

* moved custom event name to saveBaseEvent

* updated tests

* added defaults for customer field in saveEvent

* test commit

* refactor fields

* consistent naming for fields variables

* removed cartFields from saveCartEvent

* removed less important saveOrder fields

* updated event names

* moved event name field from common fields

* fixed field name

* refactor event fields

* updated generated types

* updated tests

* fixed checkout fields

* updated payload transformation functions

* removed custom event preset

* fixed identifiers transformation

* updated tests

* wrapped cart and checkout fields into objects

* updated tests

* minor changes to presets

---------

Co-authored-by: Victor Ruban <[email protected]>
Co-authored-by: Joe Ayoub <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet