Skip to content

Mc public 17 2 #92

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

Closed
wants to merge 32 commits into from
Closed

Mc public 17 2 #92

wants to merge 32 commits into from

Conversation

skovalenk
Copy link
Contributor

Description (*)

Integration tests were fixed.
ExpectationMessage and ExpectationException were fixed

Related Pull Requests

[https://github.com//issues/63](Store Front)

Fixed Issues (if relevant)

[https://github.com//issues/55](Customer rests)
[https://github.com//issues/65](Version CMS)

Contribution checklist (*)

  • Pull request has a meaningful description of its purpose
  • All commits are accompanied by meaningful commit messages
  • All new or changed code is covered with unit/integration tests (if applicable)
  • All automated tests passed successfully (all builds are green)

kokoc and others added 30 commits May 11, 2020 15:08
- Replaced API module with new one
- Fixed graphql argument reflection in order to be docblock agnostic
- Aligned with latest proto version
 - Compatibility with the latest Magento 2.4-develop
- Added category entity
- added category entity
- Fixed failure on website field
quote> - test stabilization
quote> - defaults for some values & type conversion
- tests stabilization
- Fixed more tests
- Fixed more tests
- Fixed wrong batching
- Fixed more tests
- Introduced generic object convertor
- fix Web-API Downloadable product tests
- Fixed bundle tests
- remove unused classes
- downloadable product
- Fixed wrong graphql message type for errors
mslabko and others added 2 commits June 10, 2020 13:12
- add composer json with module dependencies for git installation
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.

4 participants