Skip to content
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

806 make copyablechip also linkable #827

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

shrprabh
Copy link
Collaborator

This pull request includes changes to the CopyableChip component to add redirection functionality and minor updates to other files to accommodate these changes. The most important changes include renaming the component, adding optional URL handling, and updating related files.

Enhancements to CopyableChip component:

  • src/components/CopyableChip.tsx: Renamed CopyableChip to CopyableChipRedirection and added optional url prop to handle redirection. Implemented handleLaunch function to navigate to the specified URL when the new launch icon is clicked. [1] [2] [3] [4]

Updates to related files:

@shrprabh shrprabh linked an issue Jan 18, 2025 that may be closed by this pull request
@shrprabh shrprabh self-assigned this Jan 18, 2025
@shrprabh shrprabh requested a review from dsclassen January 18, 2025 04:32
@shrprabh
Copy link
Collaborator Author

We need to update the multi jobs entity to store the ObjectIds in MongoDB schema and write logic to update it when the user combines two or more jobs, similar to the way we are storing the bilbomd_uuids. This will allow us to pass it as a prop from the multi component and perform a redo.

@shrprabh
Copy link
Collaborator Author

Hello @dsclassen, Can we create an issue in the bilbomd-mongodb-schema repository (https://github.com/bl1231/bilbomd-mongodb-schema) and modify it to include a new field that stores the ObjectIds of combined jobs? This modification would facilitate the redirection process in the frontend.

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.

Make CopyableChip also linkable
1 participant