Skip to content

[Ramp up] Add tests for Products Update API#2883

Closed
nealweiMeta wants to merge 2 commits intomainfrom
products-api-products-update-tests
Closed

[Ramp up] Add tests for Products Update API#2883
nealweiMeta wants to merge 2 commits intomainfrom
products-api-products-update-tests

Conversation

@nealweiMeta
Copy link
Copy Markdown
Contributor

Changes proposed in this Pull Request:

Adding tests to Product Update action as ramp up task

Adding tests to Product Update action as ramp up task
@facebook-github-bot
Copy link
Copy Markdown
Contributor

@nealweiMeta has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

@facebook-github-bot
Copy link
Copy Markdown
Contributor

@nealweiMeta has updated the pull request. You must reimport the pull request before landing.

@facebook-github-bot
Copy link
Copy Markdown
Contributor

@nealweiMeta has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

@facebook-github-bot
Copy link
Copy Markdown
Contributor

@nealweiMeta merged this pull request in 1609f3b.

iodic pushed a commit that referenced this pull request Feb 18, 2025
Summary:
### Changes proposed in this Pull Request:

Adding tests to Product Update action as ramp up task

Pull Request resolved: #2883

Reviewed By: jczhuoMeta, vahidkay-meta

Differential Revision: D69501113

Pulled By: nealweiMeta

fbshipit-source-id: 1e4ba59b16610af65bf87237b6c7bea80cddfafe
facebook-github-bot pushed a commit that referenced this pull request Feb 25, 2025
Summary:
## Changelog:

* Add - FB product video field to add videos. Also added products sync to support the video field with Batch API by gurtejrehal in #2874
* Tweak - tests for #2874 by gurtejrehal in #2888
* Tweak - tests for Product Update action as ramp up task by nealweiMeta in #2883
* Fix - translations loading before the init hook by iodic in #2866
* Fix - Fixed feeds by requesting a feed file upload session after feed file is generated and added missing new fields to the feed file by mshymon in #2841

Pull Request resolved: #2891

Reviewed By: akpurifb

Differential Revision: D69836863

Pulled By: vahidkay-meta

fbshipit-source-id: 32408ab4b42f60f921bd599ed982933017185b5e
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants