Skip to content

Snippet injection Test update#3132

Merged
siyuniu-ms merged 43 commits into
mainfrom
snippet-injection
Jun 16, 2023
Merged

Snippet injection Test update#3132
siyuniu-ms merged 43 commits into
mainfrom
snippet-injection

Conversation

@siyuniu-ms

Copy link
Copy Markdown
Contributor

No description provided.

"percentage": 100
},
"preview": {
"javaScriptSnippet":

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

should this be "browserSdkLoader", saw your doc update?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

haha, yes, just updated it.

@heyams

heyams commented Jun 16, 2023

Copy link
Copy Markdown
Contributor

can you update the smoke test name to reflect this renaming and be consistent?

@trask trask left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@heyams let's wait to make release until this is merged

siyuniu-ms and others added 5 commits June 16, 2023 09:57
…ights/agent/internal/init/SecondEntryPoint.java

Co-authored-by: Trask Stalnaker <trask.stalnaker@gmail.com>
…ights/agent/internal/configuration/SnippetConfiguration.java

Co-authored-by: Trask Stalnaker <trask.stalnaker@gmail.com>
@siyuniu-ms siyuniu-ms merged commit 01a684c into main Jun 16, 2023
@siyuniu-ms siyuniu-ms deleted the snippet-injection branch June 16, 2023 17:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants