Skip to content

Remove filter name#112

Merged
carojeandat merged 5 commits into
mainfrom
remove-filter-name
Jun 9, 2026
Merged

Remove filter name#112
carojeandat merged 5 commits into
mainfrom
remove-filter-name

Conversation

@carojeandat

Copy link
Copy Markdown
Contributor

PR Summary

Remove filter name as it is no more used in filter-server

@coderabbitai

coderabbitai Bot commented May 5, 2026

Copy link
Copy Markdown

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 63feab93-7e88-4c73-9738-b3eb443ad3b1

📥 Commits

Reviewing files that changed from the base of the PR and between ab9a340 and 3462c58.

📒 Files selected for processing (1)
  • src/main/java/org/gridsuite/filter/identifierlistfilter/FilterAttributes.java
💤 Files with no reviewable changes (1)
  • src/main/java/org/gridsuite/filter/identifierlistfilter/FilterAttributes.java

📝 Walkthrough

Walkthrough

The FilterAttributes class has the String name field removed, reducing its data structure to contain only UUID id, Date modificationDate, FilterType type, and EquipmentType equipmentType.

Changes

Data Structure

Layer / File(s) Summary
Field Removal
src/main/java/org/gridsuite/filter/identifierlistfilter/FilterAttributes.java
The String name field is removed from FilterAttributes, leaving four fields: id, modificationDate, type, and equipmentType.
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title 'Remove filter name' directly and clearly describes the main change: removing the name field from FilterAttributes.
Description check ✅ Passed The description states the purpose of removing filter name because it is no longer used in filter-server, which relates to the changeset.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

Tip

💬 Introducing Slack Agent: The best way for teams to turn conversations into code.

Slack Agent is built on CodeRabbit's deep understanding of your code, so your team can collaborate across the entire SDLC without losing context.

  • Generate code and open pull requests
  • Plan features and break down work
  • Investigate incidents and troubleshoot customer tickets together
  • Automate recurring tasks and respond to alerts with triggers
  • Summarize progress and report instantly

Built for teams:

  • Shared memory across your entire org—no repeating context
  • Per-thread sandboxes to safely plan and execute work
  • Governance built-in—scoped access, auditability, and budget controls

One agent for your entire SDLC. Right inside Slack.

👉 Get started


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.

@sonarqubecloud

Copy link
Copy Markdown

@sonarqubecloud

sonarqubecloud Bot commented Jun 9, 2026

Copy link
Copy Markdown

@carojeandat carojeandat merged commit 64347d4 into main Jun 9, 2026
5 checks passed
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.

2 participants