documentation for observer-access#54
documentation for observer-access#54SyedaLamiaTabassum wants to merge 9 commits intothoth-tech:mainfrom
Conversation
feat : new property of observer-access added for read only access
✅ Deploy Preview for ontrackdocumentation ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
lachlan-robinson
left a comment
There was a problem hiding this comment.
Hi @SyedaLamiaTabassum, well done on this documentation task.
Well done:
- The documentation clearly explains the purpose and implementation of the Observer role, including its read-only policy.
- Sections are logically organized: Overview, Implementation, Database Changes, Code Logic, Unit Tests, and Conclusion.
- Code snippets for migration, authorization logic, and unit tests are provided, making it easy for developers to understand and replicate.
Consider changes:
- Minor typos (e.g., "utfmb3" should be "utf8mb3")—ensure terminology matches actual code/database settings.
- Consider using bullet points or numbered lists for steps and actions for easier readability.
This documentation is thorough and professional, providing all necessary details for understanding and maintaining the Observer role in the Doubtfire API. Minor formatting and consistency improvements could further enhance clarity.
There was a problem hiding this comment.
Well done on implementing the requested changes to the documentation. However,
- Your PR is not currently passing the automated build actions.
Make sure that:
- You have branched from the latest main branch that does not have build errors.
- You have tested your doubtfire-astro branch locally using
npm startto ensure it builds without errors.
Hello, I have tested my branch locally and the build is working now, |
WaelAlahamdi
left a comment
There was a problem hiding this comment.
Hi @SyedaLamiaTabassum, I have reviewed the documentation and found no issues. Everything looks clear and well-structured. Thank you for the work.
Description
new role of observer-access has been added that will provide read only access to specified user.
Type of change
Checklist
astro.config.mjs