Skip to content
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

Update metadata to API v61, add Apex Docs to code, improve help text and descriptions. #18

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

PawelWozniak
Copy link

I have implemented Cache Manager in my organization therefore it went through code review and was aligned to the current API and I have added as many descriptions as I could. I want to contribute with that changes to your project so others can easily understand how it works.
Changes:
Add Apex Docs to CacheManager.cls based on similar Nebula logger class and own understanding
Renamed PlatformCacheType to PlatformCachePartitionType in CacheManager.cls to align it with Nebula Logger class.
Add debug messages in case cache partition is not found to help in case of debugging.
Add description to cache CacheManagerPartition.cachePartition-meta.xml
Changed package version to 1.0.3
Updated Readme.md by required and optional configuration after installation and how to diagnose cache.
Add help text and descriptions based on Salesforce documentation lecture.

Add Apex Docs to CacheManager.cls based on similar Nebula logger class and own understanding
Renamed PlatformCacheType to PlatformCachePartitionType in CacheManager.cls to align it with Nebula Logger class.
Add debug messages in case cache partition is not found to help in case of debugging.
Add description to cache CacheManagerPartition.cachePartition-meta.xml
Changed package version to 1.0.3
…cumentation

Improved README.md by description of required and optional setup and mentioning diagnostics option.
@PawelWozniak
Copy link
Author

@jongpie Please review my changes.

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