Skip to content

Issue#253524 Feat: Add search with meta information for program-users#692

Open
Sachintechjoomla wants to merge 92 commits intoELEVATE-Project:developfrom
Sachintechjoomla:Issue#253524
Open

Issue#253524 Feat: Add search with meta information for program-users#692
Sachintechjoomla wants to merge 92 commits intoELEVATE-Project:developfrom
Sachintechjoomla:Issue#253524

Conversation

@Sachintechjoomla
Copy link

Description

These recommendations are intended to promote code quality and team communication during software development. They cover a variety of topics, including ensuring that pull requests are submitted to the correct branch, documenting new methods, preserving consistency across many services, and avoiding typical blunders like accessing APIs or DB queries within loops. Sensitive data should not be uploaded, and changes to environment variables or database models should be executed consistently. Teams may work more effectively and develop higher-quality software by adhering to these standards.

Type of change

Please choose appropriate options.

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Enhancement (additive changes to improve performance)
  • This change requires a documentation update

Checklist

  • It's critical to avoid making needless file modifications in contributions, such as adding new lines, console logs, or additional spaces, to guarantee cleaner and more efficient code. Furthermore, eliminating unnecessary imports from a file might enhance code readability and efficiency.
  • Ensure that the pull request is assigned to the right base branch and that the development branch name contains the JIRA Task Id. Furthermore, each commit message should include the JIRA Task Id in the manner "ED-100: message".
  • Only update packages if it is mentioned and authorized in the design document, and make sure that you have the required permissions.
  • Avoid making API and database queries inside a loop as it can lead to performance issues and slow down the system.
  • When calling another function inside a given function, add comments explaining the purpose and meaning of the passed arguments and expected return values.
  • If adding a blank argument in a function, add a comment explaining the reason for the blank argument.
  • Before submitting a pull request, do a self-review of your code to ensure there are no conflicts with the base branch and all comments have been addressed.
  • Before merging a pull request, it's important to have other team members review it to catch any potential errors or issues
  • To maintain code integrity, it's important to remove all related changes when removing code during a code review.
  • If new constants, endpoints, or utility functions are introduced, it is important to check if they already exist in the service to avoid any duplication.
  • Whenever a new environment variable is added to a service, it's important to ensure that the necessary changes are made to related files such as ".env.sample" and "envVariables.js" to maintain consistency and avoid errors. Additionally, the new environment variable should be added to the devops repository to ensure that it is properly documented and accessible to the team.
  • When adding a new function to a service, it is important to document it with relevant information such as the name, parameters, and return value in a consistent format across all services. Additionally, if there are any changes to the API response, ensure that the documentation in the controllers is updated accordingly.
  • Write a clear and concise commit message that describes the changes made.
  • Maintain consistent function signature and code across all services when adding a function to multiple services. Implement changes to database models in all services that use the same model.
  • Use only let and const. Do not use var.
  • Make common functions for repetitive code blocks.
  • Avoid uploading sensitive information such as secret tokens or passwords in pull requests to ensure data security.
  • Maintain consistent indentation and spacing throughout the code.

vaivk369 and others added 30 commits January 8, 2026 05:50
Issue #251045 feat: Hierarchical Categories Implementation
Export AWS_REGION and AWS_ACCOUNT_ID variables explicitly before using them in the script.
…is thrown in entity helper as tenant_id is required
…is thrown in entity helper as tenant_id is required
Issue #000 fix: if entityId is passed to the importFromLibray, error
Issue #000 fix: ProgramUser mapping
vaivk369 and others added 29 commits January 28, 2026 14:33
Issue#253042 Fix: Update task status for observation type task complete
Issue#253202 Feat:Functionality for create project from template exclude selected task
Issue#233138 Fix: Replace import from pdf function to the detailsv2
Issue#253202 Feat: userId is not explicitly provided in the request
Issue #000 fix: assign users and get users who are not yet assigned t…
Issue #000 fix: assign users and get users who are not yet assigned t…
Issue #000 fix: assign users and get users who are not yet assigned t…
…ojects

Issue #000 fix: only one onboarding project created for all participants
…ojects

Issue #000 fix: account seach error
…ojects

Issue #000 fix: duplicate function
…ojects

Issue #000 fix: entity updates should have objectId for the Ids refered
…ojects

Issue #000 fix: solution details for the IDP is not found
@coderabbitai
Copy link

coderabbitai bot commented Feb 11, 2026

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Tip

Issue Planner is now in beta. Read the docs and try it out! Share your feedback on Discord.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

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.

3 participants