-
-
Notifications
You must be signed in to change notification settings - Fork 750
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
Pr template checklist #2453
Pr template checklist #2453
Conversation
* Remove unused code and reposition create action item button * Add action item related fields to en.json and adjust column widths in ActionItemsContainer.tsx
…lity (PalisadoesFoundation#1860) * bugfix-1662 * fixed failing tests * removed allotedRoom/Seat from tableRow * bugfix-1832 * index on bugfix-1832: b584770 bugfix-1832 * added email as unique identifier * added email as unique identifier * changed column name * added translations * added translation for user --------- Co-authored-by: krishna <[email protected]>
* Removed `jest-enzyme` package * Adding check for * Added CommunityProfile page * Added tests for CommunityProfile page * Fixed formatting errors * Format fix * Merge branch 'develop' of https://github.com/git-init-priyanshu/talawa-admin-clone into priyanshu * Fixed merge conflicts * Format fix * Fix typecheck error * Added language support * Lint error fix * Integrated APIs * Revert changes * Fixed Failing Test * FIxed failing test * Fixed LoginPage failing tests * Full code coverage * Redesigned the Donations page * Added Tests * Fixed formatting errors * 100% code coverage for OrgListCard.tsx
* req shift * superadmin can view requests * fix tests * fix tests * fixed major variable issue in routeReducer and fixed test * Fix console.log statements in routesReducer * Add new mocks for organization connection and membership requests in Requests screen * Fix formatting issues and remove unused code in Requests.tsx * Fix formatting issues and remove unused code in Requests.tsx * change order of tab * fix tests
* Redesign & Refac Venue Screen * Add Tests for VenueModal * Add Test for OrganizationVenues * Migrate to card layout * refac tests for card layout * Separate VenueCard
* add tests * add tests * fix type and remove userType * remove userType * remove userType * fix type and userType * code cov * Fix async issue in Events.test.tsx * code cov for Users
…esFoundation#1875) * solved * solved * lint fixed * lint fixed2 * errors solved * test added * test added for join now and joined * test added for join now
…sFoundation#1882) * fix: fixed likes and comments for user posts * added tests * Chnages
…the talawa Admin portal (PalisadoesFoundation#1895) * redesign action items and added tests * added missing files * added missing files * added missing files * Correct changed files * fixed spelling mistake * udated delete modal and added tests * fixed linting errors * tests * fixed linting * Changed headings and added notes field * fixed linting
…sadoesFoundation#1903) - Adjusted the left drawer component to function properly on screens with a width of 820px or less. - Ensured smooth expansion and collapse of the left drawer when clicked. - Ensured the overflow-y of content is handled in smaller screens. - Modified the changes for organization dashboard screens to match the superadmin screens when the left drawer is collapsed. - Covered the changes with tests. - Ensured no other functionality got affected. Signed-off-by: Akhilender <[email protected]>
PalisadoesFoundation#1906) * Redesigned the Event Details Modal for the Talwa User and Admin portal * Ensured Creator has Event Admin capabilities
* revamped * user-event-revamped * minor
…isadoesFoundation#1901) * Updated get advertisements query and implemented infinite scroll * fix delete advertisement * improve code coverage * fix: infinite scroll * improved code coverage * removed unwanted code
…oundation#1914) * add new query * Implement Sort/Search & Add tests
* modify events query * minor corrections * modify interfaces and props * fix tests * modify code as per the new implementation * adjustments and functionality for updating single event into a recurring event * update custom recurrence rule modal * fix event modal's date and time inputs * move custom recurrence modal into recurrence options * refactor and add recurring event update functionalities * use refetch query instead of window reload * fix tests * add tests for updating recurring events * fix failing tests
* table re-design * add delete btn * Update localization files with new translations and add delete button in CampaignUpdateModal.tsx * fix tests * Refactor CampaignUpdateModal.tsx and OrganizationFundCampagins.tsx * Refactor OrganizationFundCampagins.tsx: Remove commented out code for delete button * Refactor CampaignUpdateModal.tsx: Remove unused imports and update interface import
…oundation#1911) * Yearly Calender Screens * updated user screen * user-calender-view * locales files updated * fixed linting errors * added tests and changed font
…alisadoesFoundation#1920) * regular user can access the dashboard of the event created by them * fix the test to have 100% coverage
* fixed people screen filter * fixed the OrgAdminListCard test
* updated User side bar * improve code coverage * added profile dropdown * removed commented code * fix user sidebar on event dashboard screen * added translations * fix: failing tests * added visit button to navigate to organizations
…ndation#1935) Bumps [@types/node](https://github.com/DefinitelyTyped/DefinitelyTyped/tree/HEAD/types/node) from 12.20.55 to 20.12.7. - [Release notes](https://github.com/DefinitelyTyped/DefinitelyTyped/releases) - [Commits](https://github.com/DefinitelyTyped/DefinitelyTyped/commits/HEAD/types/node) --- updated-dependencies: - dependency-name: "@types/node" dependency-type: direct:development update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…ndation#1937) Bumps [@types/js-cookie](https://github.com/DefinitelyTyped/DefinitelyTyped/tree/HEAD/types/js-cookie) from 3.0.3 to 3.0.6. - [Release notes](https://github.com/DefinitelyTyped/DefinitelyTyped/releases) - [Commits](https://github.com/DefinitelyTyped/DefinitelyTyped/commits/HEAD/types/js-cookie) --- updated-dependencies: - dependency-name: "@types/js-cookie" dependency-type: direct:development update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…doesFoundation#1943) Bumps [eslint-config-prettier](https://github.com/prettier/eslint-config-prettier) from 8.10.0 to 9.1.0. - [Changelog](https://github.com/prettier/eslint-config-prettier/blob/main/CHANGELOG.md) - [Commits](prettier/eslint-config-prettier@v8.10.0...v9.1.0) --- updated-dependencies: - dependency-name: eslint-config-prettier dependency-type: direct:development update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…isadoesFoundation#2404) Bumps [@babel/preset-typescript](https://github.com/babel/babel/tree/HEAD/packages/babel-preset-typescript) from 7.24.7 to 7.26.0. - [Release notes](https://github.com/babel/babel/releases) - [Changelog](https://github.com/babel/babel/blob/main/CHANGELOG.md) - [Commits](https://github.com/babel/babel/commits/v7.26.0/packages/babel-preset-typescript) --- updated-dependencies: - dependency-name: "@babel/preset-typescript" dependency-type: direct:development update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…isadoesFoundation#2407) Bumps [@types/react-bootstrap](https://github.com/DefinitelyTyped/DefinitelyTyped/tree/HEAD/types/react-bootstrap) from 0.32.32 to 0.32.37. - [Release notes](https://github.com/DefinitelyTyped/DefinitelyTyped/releases) - [Commits](https://github.com/DefinitelyTyped/DefinitelyTyped/commits/HEAD/types/react-bootstrap) --- updated-dependencies: - dependency-name: "@types/react-bootstrap" dependency-type: direct:development update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…ndation#2403) Bumps [@apollo/client](https://github.com/apollographql/apollo-client) from 3.11.4 to 3.11.8. - [Release notes](https://github.com/apollographql/apollo-client/releases) - [Changelog](https://github.com/apollographql/apollo-client/blob/main/CHANGELOG.md) - [Commits](apollographql/apollo-client@v3.11.4...v3.11.8) --- updated-dependencies: - dependency-name: "@apollo/client" dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* Update EventCalendar.tsx * Update EventCalendar.test.tsx
* changed color scheme for the organization people screen * fix precommit * merge * Update pre-commit * fix conflicts * fix type checks * fix type checks * fix type checks * fix ts eslint errors * fix ts eslint errors * fix ts eslint errors * fix ts eslint errors * testing * testing * testing * reverted changes in yaml file * cr comments * Update pull-request.yml * cr comments * cr comments and single css file * CR comments * delete button margin from top * prettier for commit and pull request * remove hard coded colors * fix failing test cases
…undation#2388) * major changes event attendence stastics * add bar chart adn recurring event count * major chart and ui fixes * modal sizing and filtering * “event” * charts and recurring bugs * member details ui revamped * tested member details functionalities * added eventsattended in user portal * added test files * added tests for eventstatics,eventdashboard and related components * added tests for Attendance and Statistics * suggestion added in files * Update src/components/EventRegistrantsModal/EventRegistrantsModal.tsx Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com> * error handling and short lines of code * changes added * added null checks * refactored code * refactoring * improved refactoring statistics * minor * some iterations * package changes * minor changes * minor * added translations and refactoring * added translations and refactoring. * Added tsdoc comments * Added more docs * Added missed translations * .. * Revert ".." This reverts commit ed410c7. * .. * minor files * minor * added testing and schema * minor * Added changes * type fixes * added test for eventstats and eventattendance * minor formatting * add test cases * added registrants test cases * added orgs testcases * fixed orgdash change * increased code coverage * increased dateformatter files coverage * minor change --------- Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
…on#2350) * Added scripts for talawa websocket url * Fixed naming inconsistency Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com> * Fixed naming inconsistencies * Added error handling Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com> * Improved test coverage * Fixed scripts for websocketurl * updated INSTALLATION.md * Updated logic to handle duplicate entries * Fixed errors * Added tests * Undo changes to .env.example * Chore/organization people UI changes (PalisadoesFoundation#2358) * changed color scheme for the organization people screen * fix precommit * merge * Update pre-commit * fix conflicts * fix type checks * fix type checks * fix type checks * fix ts eslint errors * fix ts eslint errors * fix ts eslint errors * fix ts eslint errors * testing * testing * testing * reverted changes in yaml file * cr comments * Update pull-request.yml * cr comments * cr comments and single css file * CR comments * delete button margin from top * prettier for commit and pull request * remove hard coded colors * fix failing test cases * Upgraded dicebear (PalisadoesFoundation#2411) --------- Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com> Co-authored-by: Peter Harrison <[email protected]> Co-authored-by: ANKIT VARSHNEY <[email protected]>
…oundation#2419) Bumps [@types/react-dom](https://github.com/DefinitelyTyped/DefinitelyTyped/tree/HEAD/types/react-dom) from 18.3.0 to 18.3.1. - [Release notes](https://github.com/DefinitelyTyped/DefinitelyTyped/releases) - [Commits](https://github.com/DefinitelyTyped/DefinitelyTyped/commits/HEAD/types/react-dom) --- updated-dependencies: - dependency-name: "@types/react-dom" dependency-type: direct:development update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…Foundation#2420) Bumps [vite-tsconfig-paths](https://github.com/aleclarson/vite-tsconfig-paths) from 5.0.1 to 5.1.2. - [Release notes](https://github.com/aleclarson/vite-tsconfig-paths/releases) - [Commits](aleclarson/vite-tsconfig-paths@v5.0.1...v5.1.2) --- updated-dependencies: - dependency-name: vite-tsconfig-paths dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…ndation#2421) Bumps [@pdfme/generator](https://github.com/pdfme/pdfme) from 5.0.0 to 5.1.7. - [Release notes](https://github.com/pdfme/pdfme/releases) - [Changelog](https://github.com/pdfme/pdfme/blob/main/RELEASE.md) - [Commits](pdfme/pdfme@5.0.0...5.1.7) --- updated-dependencies: - dependency-name: "@pdfme/generator" dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [sass](https://github.com/sass/dart-sass) from 1.77.8 to 1.80.6. - [Release notes](https://github.com/sass/dart-sass/releases) - [Changelog](https://github.com/sass/dart-sass/blob/main/CHANGELOG.md) - [Commits](sass/dart-sass@1.77.8...1.80.6) --- updated-dependencies: - dependency-name: sass dependency-type: direct:development update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…sFoundation#2425) * show tags assigned to a user * add tests * fix translations
…undation#2417) * changed color schema for the tag and action item screen * subtag screen color schema changes * changes
Bumps [sass](https://github.com/sass/dart-sass) from 1.80.6 to 1.80.7. - [Release notes](https://github.com/sass/dart-sass/releases) - [Changelog](https://github.com/sass/dart-sass/blob/main/CHANGELOG.md) - [Commits](sass/dart-sass@1.80.6...1.80.7) --- updated-dependencies: - dependency-name: sass dependency-type: direct:development update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…oesFoundation#2434) Bumps [eslint-plugin-import](https://github.com/import-js/eslint-plugin-import) from 2.30.0 to 2.31.0. - [Release notes](https://github.com/import-js/eslint-plugin-import/releases) - [Changelog](https://github.com/import-js/eslint-plugin-import/blob/main/CHANGELOG.md) - [Commits](import-js/eslint-plugin-import@v2.30.0...v2.31.0) --- updated-dependencies: - dependency-name: eslint-plugin-import dependency-type: direct:development update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…dation#2435) Bumps [@mui/x-charts](https://github.com/mui/mui-x/tree/HEAD/packages/x-charts) from 7.22.1 to 7.22.2. - [Release notes](https://github.com/mui/mui-x/releases) - [Changelog](https://github.com/mui/mui-x/blob/v7.22.2/CHANGELOG.md) - [Commits](https://github.com/mui/mui-x/commits/v7.22.2/packages/x-charts) --- updated-dependencies: - dependency-name: "@mui/x-charts" dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…dation#2436) Bumps [@types/react](https://github.com/DefinitelyTyped/DefinitelyTyped/tree/HEAD/types/react) from 18.3.3 to 18.3.12. - [Release notes](https://github.com/DefinitelyTyped/DefinitelyTyped/releases) - [Commits](https://github.com/DefinitelyTyped/DefinitelyTyped/commits/HEAD/types/react) --- updated-dependencies: - dependency-name: "@types/react" dependency-type: direct:development update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Our Pull Request Approval ProcessWe have these basic policies to make the approval process smoother for our volunteer team. Testing Your CodePlease make sure your code passes all tests. Our test code coverage system will fail if these conditions occur:
The process helps maintain the overall reliability of the code base and is a prerequisite for getting your PR approved. Assigned reviewers regularly review the PR queue and tend to focus on PRs that are passing. 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
|
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.
Congratulations on making your first PR! 🎊 If you haven't already, check out our Contributing Guidelines and PR Reporting Guidelines to ensure that you are following our guidelines for contributing and creating PR.
Note Currently processing new changes in this PR. This may take a few minutes, please wait... ⛔ Files ignored due to path filters (75)
📒 Files selected for processing (82)
Tip CodeRabbit can enforce grammar and style rules using `languagetool`.Configure 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
|
What kind of change does this PR introduce?
Updated pr template which consists test checklist .
Issue Number:
Fixes #2448
Did you add tests for your changes?
No
Snapshots/Videos:
If relevant, did you update the documentation?
Yes - This PR is a documentation update
Summary
Updated the PR template to include:
Checklist for implementing coderaabbitai suggestions
Checklist for verifying test coverage requirements (~90/95%)
Does this PR introduce a breaking change?
No
Other information
Have you read the contributing guide?