-
-
Notifications
You must be signed in to change notification settings - Fork 869
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
Feature/admin docs gen #3259
Feature/admin docs gen #3259
Conversation
Important Review skippedMore than 25% of the files skipped due to max files limit. The review is being skipped to prevent a low-quality review. 218 files out of 300 files are above the max files limit of 75. Please upgrade to Pro plan to get higher limits. You can disable this status message by setting the Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
Documentation and Community
|
Our Pull Request Approval ProcessThanks for contributing! Testing Your CodeRemember, your PRs won't be reviewed until these criteria are met:
Our policies make our code better. ReviewersDo not assign reviewers. Our Queue Monitors will review your PR and assign them.
Reviewing Your CodeYour reviewer(s) will have the following roles:
CONTRIBUTING.mdRead our CONTRIBUTING.md file. Most importantly:
Other
|
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## develop-postgres #3259 +/- ##
=====================================================
+ Coverage 8.53% 89.82% +81.29%
=====================================================
Files 309 331 +22
Lines 7976 8583 +607
Branches 1766 1901 +135
=====================================================
+ Hits 681 7710 +7029
+ Misses 7218 612 -6606
- Partials 77 261 +184
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
@bint-Eve @tasneemkoushar
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please work on doing the merge in the PR not the Push GitHub Workflow.
* implement a Visit button in the Joined Organizations filter * Resolved: Potential Issue Identified by CodeRabbit AI * Resolved: Potential Issue Identified by CodeRabbit AI * Resolved: Potential Issue Identified by CodeRabbit AI * Resolved: Potential Issue Identified by CodeRabbit AI * Update src/components/OrganizationCard/OrganizationCard.tsx Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com> * Added test cases * Added test cases * added tsdoc * resolve: all issue --------- Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
…3277) * Update Contributing.md * Fixed formatting * Fix linting error * updated eslintignore to exclude markdown files * Implemented requested changes * Removed numbering
* changed green to blue * updated funds section * people section css updated * updated Tags section * venues, action items, advertisments, plugins sections updated * fixed code coverage test * fixed code coverage test * CSS methodology added * fixed linting --------- Co-authored-by: Rahul Chougule <[email protected]>
@bint-Eve there are couple of changes in the src folder. If this is some other issue, can you extract those changes to a separate PR please. |
* Fixed error message on PostCard.tsx * Added translation of text and added a test case for my code * handle empty comment validation * updated error handling * improved error handling * Improve error handling and add test coverage for edge cases * Added missing providers
* Improved Code Coverage of UserListCard.tsx * Updated Code Rabbit suggestions and fixed the failing test case * Fixed Few CodeRabbit Suggestions
* chore: fixed node warning and deprecation msgs * trigger ci: rerun pipeline
Please fix the conflicting file |
Merge with the latest upstream and reinstall packages |
@bint-Eve we want to merge your PR, can you please resolve conflicts. |
@bint-Eve you need to give write permission to the workflow as well, only then it is suppose to do the commit to the branch. |
@gautam-divyanshu tried that without success. He feels the reason why that PR passed was that there was nothing to commit. |
* added test cases in src/screens/Users/Users.spec.tsx * fixed linting issues * fixed linting issues * fixed linting issues * added test case to improve coverage in the Organization.mocks.ts file * improved coverage in Users.tsx
* improve code coverage of people.tsx * fixed formating * added coderabit suggetions * removed a comment * removed redundant comment * added code rabbit's suggetions * added code rabbit's suggetions
* auto-docs * yarn to npm * broken links fixed and code refactored * github action created * pull-request.yml updated * pull-request.yml updated * pull-request.yml updated * pull-request.yml updated * pull-request.yml updated * pull-request.yml updated * pull-request.yml updated * pull-request.yml updated * pull-request.yml updated * pull-request.yml updated * pull-request.yml updated * pull-request.yml updated * pull-request.yml updated * pull-request.yml updated * pull-request.yml updated * pull-request.yml updated * pull-request.yml updated * docs removed * pull-request.yml updated * pull-request.yml updated * pull-request.yml updated * pull-request.yml updated * pull-request.yml updated * pull-request.yml updated * pull-request.yml updated * pull-request.yml updated * pull-request.yml updated * pull-request.yml updated * pull-request.yml updated * github action removed & husky used * auto docs in gitignore * pull-request.yml updated * pull-request.yml updated * pull-request.yml updated * pull-request.yml updated * pull-request.yml updated * pull-request.yml updated
* Code coverage improvement * Linting issue resolved * Reduced the no of ignore comments by adding more tests * Reduce no of lines in OrganizationTagsMocks.ts * Deleted MOCKS_EMPTY_USER_TAG * Added appropiate error message * Resolving conflicting files * Removed all ignore statements
* Updated documentation layout * Removed outdated sidebar
* Updated documentation layout * Removed outdated sidebar * Minor change to test website deployment
… jest to vitest (#3355)
… proper casing to completed ads section (#3362)
* ui fix * fix * css fix * Update src/screens/UserPortal/People/People.tsx Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com> * fix * fix * Update src/style/app.module.css Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com> * add test and ui fix * fix * ui fix * fix * ui fix * ui fix --------- Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
closing this PR as this task is done in another PR |
What kind of change does this PR introduce?
This PR solves the second part of #2970 - Ensure the autogenerated markdown based on the code base's comments is added to the site in an intuitive way.
Issue Number:
Fixes #2970
Did you add tests for your changes?
No
If relevant, did you update the documentation?
NOT manually, Docs will be auto-generate by husky hooks
Summary
Documentation is structured into two parts- autogenerated documentation which is done by husky/pre-commit hook (npm auto generate-docs) and docs/auto-docs directory and manual documentation which is done by removing default docusaurus files and making 2 placeholders (user-guide and developer guide). This documentation section is accessible through general navbar item.
Does this PR introduce a breaking change?
No
Other information
Creation of new file tsconfig.docs.json was done to optimize TypeScript Compilation for Documentation. It configures TypeScript to handle files in a way that aligns with the documentation generation process. tsconfig.json might have stricter rules (e.g., strict: true), which aren't necessary for docs and were hindering doc gen unnecessarily.
Have you read the contributing guide?
Yes