Skip to content

Adds MetadataManager - #27

Merged
huangdijia merged 4 commits into
2.xfrom
2.x-metadata-manager
Aug 14, 2024
Merged

Adds MetadataManager#27
huangdijia merged 4 commits into
2.xfrom
2.x-metadata-manager

Conversation

@huangdijia

Copy link
Copy Markdown
Contributor

No description provided.

… and retrieval

The code changes in `MetadataManager.php` remove the unnecessary comments and update the `register` and `get` methods to be static. This refactor improves the code readability and maintainability by clearly indicating that these methods can be accessed directly on the class without needing an instance.

Co-authored-by: Deeka Wong <8337659+huangdijia@users.noreply.github.com>
The code changes in `ClientTest.php` update the `ClientFactory::create()` method to pass the `this->service` parameter as the first argument instead of `'test'`. This change ensures that the correct service is used when creating a new instance of the client.
@huangdijia
huangdijia marked this pull request as ready for review August 14, 2024 03:40
@huangdijia
huangdijia merged commit 2547b65 into 2.x Aug 14, 2024
@huangdijia
huangdijia deleted the 2.x-metadata-manager branch August 14, 2024 03:40
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.

1 participant