Skip to content

[FSSDK-8837] Implement API for sendOdpEvent in Agent#356

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
Merged
Mat001 merged 15 commits into from
Mar 7, 2023
Merged

[FSSDK-8837] Implement API for sendOdpEvent in Agent #356

Mat001 merged 15 commits into from
Mar 7, 2023

Conversation

@Mat001
Copy link
Contributor

@Mat001Mat001 commented Feb 24, 2023•
edited
Loading

Summary

  • Implement API for sendOdpEvent, update open api schema. Extend agent config to include odp.
  • extend config to accommodate ODP
  • update open API schema
  • unit tests

Issues

@Mat001Mat001 self-assigned this Feb 24, 2023
@Mat001Mat001 changed the title [FSSDK-8837] Add ODP wrapper[FSSDK-8837] Extend Agent with ODP wrapper functionalityFeb 25, 2023
@Mat001Mat001 changed the title [FSSDK-8837] Extend Agent with ODP wrapper functionality[FSSDK-8837] Extend Agent with ODP functionalityFeb 25, 2023
@Mat001Mat001 changed the title [FSSDK-8837] Extend Agent with ODP functionality[FSSDK-8837] Implement API for sendOdpEvent in AgentMar 4, 2023
jaeopt
jaeopt requested changes Mar 6, 2023
View reviewed changes
Copy link
Contributor

@jaeoptjaeopt left a comment

Choose a reason for hiding this comment

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

Looks good. A few changes suggested.

@Mat001Mat001 requested a review from jaeoptMarch 7, 2023 19:16
yasirfolio3
yasirfolio3 approved these changes Mar 7, 2023
View reviewed changes
Copy link
Contributor

@yasirfolio3yasirfolio3 left a comment

Choose a reason for hiding this comment

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

1 minor change suggested, looks good overall.

jaeopt
jaeopt approved these changes Mar 7, 2023
View reviewed changes
Copy link
Contributor

@jaeoptjaeopt left a comment

Choose a reason for hiding this comment

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

LGTM

@Mat001Mat001 merged commit e2f365c into masterMar 7, 2023
@Mat001Mat001 deleted the mpirnovar-odp branch March 7, 2023 19:45
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jaeoptjaeoptjaeopt approved these changes

@andrewleap-optimizelyandrewleap-optimizelyAwaiting requested review from andrewleap-optimizely

+1 more reviewer

@yasirfolio3yasirfolio3yasirfolio3 approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@Mat001Mat001

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@Mat001@yasirfolio3@jaeopt