Skip to content

Feature/epmrpp 104081 merge orgs to dev #4401

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

Merged
merged 328 commits into from
Jun 12, 2025

Conversation

BlazarQSO
Copy link
Contributor

@BlazarQSO BlazarQSO commented Jun 11, 2025

PR Checklist

  • Have you verified that the PR is pointing to the correct target branch? (develop for features/bugfixes, other if mentioned in the task)
  • Have you verified that your branch is consistent with the target branch and has no conflicts? (if not, make a rebase under the target branch)
  • Have you checked that everything works within the branch according to the task description and tested it locally?
  • Have you run the linter (npm run lint) prior to submission? Enable the git hook on commit in your IDE to run it and format the code automatically.
  • Have you run the tests locally and added/updated them if needed?
  • Have you checked that app can be built (npm run build)?
  • Have you checked that no new circular dependencies appreared with your changes? (the webpack plugin reports circular dependencies within the dev npm script)
  • Have you made sure that all the necessary pipelines has been successfully completed?
  • If the task requires translations to be updated, have you done this by running the manage:translations script?
  • Have you added the link to the PR in the Jira ticket comments?

Visuals

BlazarQSO and others added 30 commits July 18, 2024 17:49
…-develop-to-org

Feature/epmrpp 92632 merge develop to org
* EPMRPP-77768 || Sidebar redesign. Instance level

* EPMRPP-77768 || fix eslint

* EPMRPP-77768 || Code Review fix - 1

* EPMRPP-77768 || remove sidebarType

* EPMRPP-77768 || Code Review fix - 2

* EPMRPP-77768 || Code Review fix - 3
#3936)

* EPMRPP-92591 || Extra Organization request is sent when opening Projects page via sidebar

* EPMRPP-92591 || Code Review fix - 1
#3938)

* EPMRPP-92683 || Sidebar improvements for organizations with no assignments

* EPMRPP-92683 || Code Review fix - 1

* EPMRPP-92683 || Code Review fix-2

* EPMRPP-92683 || Code Review fix - 3

* EPMRPP-92683 || Code Review fix - 4
#3939)

* EPMRPP-93205 || Wrong behaviour when clicking on All organizations button in the sidebar

* EPMRPP-93205 || Code Review fix - 1

* EPMRPP-93205 || Code Review fix - 2
* EPMRPP-93200 || Wrong Sidebar level is displayed when opening Profile page

* EPMRPP-93200 || Fix routes overlapping

* EPMRPP-93200 || fix the links in the organization sidebar

---------

Co-authored-by: Ilya Hancharyk <[email protected]>
…-develop-to-orgs

Feature/epmrpp 93163 merge develop to orgs
* EPMRPP-90346 || Integrate pagination component

* EPMRPP-90346 || Code Review fix - 1

* EPMRPP-90346 || Code Review fix - 2

* EPMRPP-90346 || fix projects reducer

* EPMRPP-90346 || fix conditions

* EPMRPP-90346 || add default properties

* EPMRPP-90346 || Code Review fix - 3

* EPMRPP-90346 || Code Reiview fix - 4

* EPMRPP-90346 || Code Review fix - 5

* EPMRPP-90346 || Code Review fix - 6

* EPMRPP-90346 || Code Review fix - 7

* EPMRPP-90346 || fix styles

* EPMRPP-90346 || Code Review fix - 7
* EPMRPP-91199 || Search for organizations and projects

* EPMRPP-91199 || fix case insensitive search

* EPMRPP-92480 || code review fix - 1

* EPMRPP-92480 || code review fix - 2

* EPMRPP-92480 || ui-kit version update

---------

Co-authored-by: maria-hambardzumian <[email protected]>
…-to-orgs

Feature/epmrpp 93592 merge to orgs
* EPMRPP-93360 || Invite user modal

* EPMRPP-93360 || Code Review fix - 1

* EPMRPP-93360 || Code Review fix - 2
…#3977)

* EPMRPP-93769 ||  Left and right arrows stay highlighted after clicking on them

* EPMRPP-93769 || fix build
* EPMRPP-92701 || Create project and Invite user buttons have broken styles

* EPMRPP-92701 || import icons from the ui kit

* EPMRPP-92701 || update ui-kit

* EPMRPP-92701 || fix button property
* EPMRPP-89679 || Handle project creation in Redux

* EPMRPP-89679 || Update design of AddProjectModal

* EPMRPP-89679 || Update and refine validators

* EPMRPP-89679 || Integrate AddProjectModal into Projects page

* EPMRPP-89679 || Update messages

* EPMRPP-89679 || test fix

* EPMRPP-89679 || translations

* EPMRPP-89679 || code review fix - 1

* EPMRPP-89679 || hint fix
…-dev-to-orgs

Feature/epmrpp 94506 merge dev to orgs
BlazarQSO and others added 16 commits May 27, 2025 16:06
* EPMRPP-103709 || Wrong parameter value is sent for not contains

* EPMRPP-103709 || change name
…al pages (#4382)

* EPMRPP-103727 || Users names are not displayed in the tables in several pages

* EPMRPP-103727 || Code review fixes - 1

* EPMRPP-103727 || Code review fixes - 1

* EPMRPP-103727 || Code review fixes - 1
init tsconfig file for ts migration configuration
…e-dev-to-orgs

Feature/epmrpp 103708 merge dev to orgs
* EPMRPP-103422 || Org settings page layout (#4370)

* EPMRPP-103422 || Org settings page layout

* EPMRPP-103422 || Code Review fixes - 1

* EPMRPP-103423 || General tab (#4380)

* EPMRPP-103423 || General tab

* EPMRPP-103423 || General tab fixes

* EPMRPP-103423 || Code review fixes - 1

* EPMRPP-103423 || Implement request to get organization settings

* EPMRPP-103423 || Code review fixes - 2


Co-authored-by: Alla Prishchepa <[email protected]>
#4392)

* EPMRPP-104133 || Wrong placeholder and extra asterisk is displayed in the modal

* EPMRPP-104133 || Code Review fix - 1
)

* EPMRPP-103721 || Remove stale Dropdown in favor of ui-kit version

* EPMRPP-103721 || Remove redundant fragment

* EPMRPP-103721 || Update AnalyzerMode dropdown

---------

Co-authored-by: Alla Prishchepa <[email protected]>
Copy link

coderabbitai bot commented Jun 11, 2025

Important

Review skipped

More than 25% of the files skipped due to max files limit. The review is being skipped to prevent a low-quality review.

166 files out of 281 files are above the max files limit of 100. Please upgrade to Pro plan to get higher limits.

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

✨ Finishing Touches
  • 📝 Generate Docstrings
🧪 Generate Unit Tests
  • Create PR with Unit Tests
  • Commit Unit Tests in branch feature/EPMRPP-104081-merge-orgs-to-dev
  • Post Copyable Unit Tests in Comment

🪧 Tips

Chat

There are 3 ways to chat with CodeRabbit:

‼️ IMPORTANT
Auto-reply has been disabled for this repository in the CodeRabbit settings. The CodeRabbit bot will not respond to your replies unless it is explicitly tagged.

  • Files and specific lines of code (under the "Files changed" tab): Tag @coderabbitai in a new review comment at the desired location with your query. Examples:
    • @coderabbitai explain this code block.
    • @coderabbitai modularize this function.
  • PR comments: Tag @coderabbitai in a new PR comment to ask questions about the PR branch. For the best results, please provide a very specific query, as very limited context is provided in this mode. Examples:
    • @coderabbitai gather interesting stats about this repository and render them as a table. Additionally, render a pie chart showing the language distribution in the codebase.
    • @coderabbitai read src/utils.ts and explain its main purpose.
    • @coderabbitai read the files in the src/scheduler package and generate a class diagram using mermaid and a README in the markdown format.
    • @coderabbitai help me debug CodeRabbit configuration file.

Support

Need help? Create a ticket on our support page for assistance with any issues or questions.

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)

  • @coderabbitai pause to pause the reviews on a PR.
  • @coderabbitai resume to resume the paused reviews.
  • @coderabbitai review to trigger an incremental review. This is useful when automatic reviews are disabled for the repository.
  • @coderabbitai full review to do a full review from scratch and review all the files again.
  • @coderabbitai summary to regenerate the summary of the PR.
  • @coderabbitai generate docstrings to generate docstrings for this PR.
  • @coderabbitai generate sequence diagram to generate a sequence diagram of the changes in this PR.
  • @coderabbitai auto-generate unit tests to generate unit tests for this PR.
  • @coderabbitai resolve resolve all the CodeRabbit review comments.
  • @coderabbitai configuration to show the current CodeRabbit configuration for the repository.
  • @coderabbitai help to get help.

Other keywords and placeholders

  • Add @coderabbitai ignore anywhere in the PR description to prevent this PR from being reviewed.
  • Add @coderabbitai summary to generate the high-level summary at a specific location in the PR description.
  • Add @coderabbitai anywhere in the PR title to generate the title automatically.

CodeRabbit Configuration File (.coderabbit.yaml)

  • You can programmatically configure CodeRabbit by adding a .coderabbit.yaml file to the root of your repository.
  • Please see the configuration documentation for more information.
  • If your editor has YAML language server enabled, you can add the path at the top of this file to enable auto-completion and validation: # yaml-language-server: $schema=https://coderabbit.ai/integrations/schema.v2.json

Documentation and Community

  • Visit our Documentation for detailed information on how to use CodeRabbit.
  • Join our Discord Community to get help, request features, and share feedback.
  • Follow us on X/Twitter for updates and announcements.

@codecov-commenter
Copy link

codecov-commenter commented Jun 11, 2025

Codecov Report

Attention: Patch coverage is 65.90909% with 30 lines in your changes missing coverage. Please review.

Project coverage is 66.59%. Comparing base (2d37ccf) to head (ca470c2).

Files with missing lines Patch % Lines
app/src/common/utils/permissions/getRoleTitle.js 21.42% 7 Missing and 4 partials ⚠️
...pp/src/common/utils/validation/commonValidators.js 0.00% 10 Missing ⚠️
app/src/common/constants/organizationTypes.js 0.00% 2 Missing ⚠️
app/src/common/utils/validation/asyncValidation.js 0.00% 2 Missing ⚠️
app/src/common/constants/accountType.js 0.00% 1 Missing ⚠️
...mon/constants/localization/settingsLocalization.js 0.00% 1 Missing ⚠️
app/src/common/constants/userProfileRoutes.js 0.00% 1 Missing ⚠️
app/src/common/utils/permissions/permissions.js 96.66% 1 Missing ⚠️
app/src/common/utils/validation/validate.js 87.50% 1 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #4401      +/-   ##
===========================================
- Coverage    66.99%   66.59%   -0.40%     
===========================================
  Files           81       85       +4     
  Lines          924      982      +58     
  Branches       137      140       +3     
===========================================
+ Hits           619      654      +35     
- Misses         277      297      +20     
- Partials        28       31       +3     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Copy link

sonarqubecloud bot commented Jun 12, 2025

Quality Gate Passed Quality Gate passed

Issues
0 New issues
29 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
1.4% Duplication on New Code

See analysis details on SonarQube Cloud

@BlazarQSO BlazarQSO merged commit 001658d into develop Jun 12, 2025
10 checks passed
@BlazarQSO BlazarQSO deleted the feature/EPMRPP-104081-merge-orgs-to-dev branch June 12, 2025 12:15
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.

9 participants