Skip to content

feat(client): removed duplicated options from add modals list#1286

Merged
johbaxter merged 13 commits intodevfrom
1242-Add-modal-duplicate-option-removal
Jul 7, 2025
Merged

feat(client): removed duplicated options from add modals list#1286
johbaxter merged 13 commits intodevfrom
1242-Add-modal-duplicate-option-removal

Conversation

@Paulson-Robert
Copy link
Copy Markdown
Contributor

Description

Changes Made

How to Test

  1. Steps to reproduce/test the behavior
  2. Expected outcomes

Notes

@Paulson-Robert Paulson-Robert requested a review from johbaxter June 10, 2025 06:45
@Paulson-Robert Paulson-Robert self-assigned this Jun 10, 2025
@Paulson-Robert Paulson-Robert requested a review from a team as a code owner June 10, 2025 06:45
@Paulson-Robert Paulson-Robert linked an issue Jun 10, 2025 that may be closed by this pull request
3 tasks
@github-actions
Copy link
Copy Markdown

@CodiumAI-Agent /describe

@QodoAI-Agent
Copy link
Copy Markdown

Title

feat(client): removed duplicated options from add modals list


User description

Description

Changes Made

How to Test

  1. Steps to reproduce/test the behavior
  2. Expected outcomes

Notes


PR Type

Enhancement


Description

  • Remove duplicate NeMo option entry

  • Clean up connection options list


Changes walkthrough 📝

Relevant files
Enhancement
import.constants.ts
Remove duplicate NeMo option block                                             

packages/client/src/pages/import/import.constants.ts

  • Deleted duplicate NeMo option object
  • Removed associated disable, icon, and fields
  • Ensured list consistency for import CONNECTION_OPTIONS
  • +0/-6     

    Need help?
  • Type /help how to ... in the comments thread for any questions about PR-Agent usage.
  • Check out the documentation for more information.
  • @github-actions
    Copy link
    Copy Markdown

    @CodiumAI-Agent /review

    @QodoAI-Agent
    Copy link
    Copy Markdown

    PR Reviewer Guide 🔍

    Here are some key observations to aid the review process:

    ⏱️ Estimated effort to review: 1 🔵⚪⚪⚪⚪
    🧪 No relevant tests
    🔒 No security concerns identified
    ⚡ Recommended focus areas for review

    Stale References

    Verify that removing the NeMo option does not leave any unused code paths or broken references elsewhere in the codebase.

                defaultValue: '',
                options: {
                    component: 'text-field',
                },
                disabled: false,
            },
        ],
    },
    {
        name: 'Replit code model – 3b',
        disable: false,
        icon: REPLIT,
        fields: [
            {
                fieldName: 'NAME',
                label: 'Catalog Name',
                defaultValue: '',
                options: {
                    component: 'text-field',
                },
                disabled: false,
                rules: {

    @github-actions
    Copy link
    Copy Markdown

    @CodiumAI-Agent /improve

    @QodoAI-Agent
    Copy link
    Copy Markdown

    PR Code Suggestions ✨

    No code suggestions found for the PR.

    @johbaxter johbaxter merged commit 5997902 into dev Jul 7, 2025
    3 checks passed
    @johbaxter johbaxter deleted the 1242-Add-modal-duplicate-option-removal branch July 7, 2025 13:56
    @github-actions
    Copy link
    Copy Markdown

    github-actions bot commented Jul 7, 2025

    @CodiumAI-Agent /update_changelog

    @QodoAI-Agent
    Copy link
    Copy Markdown

    Changelog updates: 🔄

    2025-07-07 *

    Changed

    • Remove duplicate options from add modals list

    to commit the new content to the CHANGELOG.md file, please type:
    '/update_changelog --pr_update_changelog.push_changelog_changes=true'

    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.

    Add Model Duplicate Option

    4 participants