Skip to content

fix(UI): Added code to resolve duplicate release. #1576

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.

Sign up for GitHub

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

Open
nikkuma7 wants to merge 1 commit into
base: main
Choose a base branch
from
Open

fix(UI): Added code to resolve duplicate release. #1576

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
10 changes: 6 additions & 4 deletions src/app/[locale]/components/detail/[id]/components/ReleaseOverview.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -147,10 +147,12 @@ const ReleaseOverview = ({ componentId, calledFromModerationRequestDetail }: Pro
</Link>
</OverlayTrigger>
<OverlayTrigger overlay={<Tooltip>{t('Duplicate')}</Tooltip>}>
<BsClipboard
className='btn-icon'
size={20}
/>
<Link href={`/components/editRelease/${id}?duplicate=true`}>
<BsClipboard
className='btn-icon'
size={20}
/>
</Link>
</OverlayTrigger>
<OverlayTrigger overlay={<Tooltip>{t('Link Project')}</Tooltip>}>
<BsLink45Deg
Expand Down
100 changes: 80 additions & 20 deletions src/app/[locale]/components/editRelease/[id]/components/EditRelease.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -59,12 +59,13 @@ const EditRelease = ({ releaseId, isSPDXFeatureEnabled }: Props): ReactNode => {
const router = useRouter()
const t = useTranslations('default')
const params = useSearchParams()
const isDuplicate = params.get('duplicate') === 'true'
const [release, setRelease] = useState<ReleaseDetail>()
const [componentId, setComponentId] = useState('')
const [deletingRelease, setDeletingRelease] = useState('')
const [deleteModalOpen, setDeleteModalOpen] = useState(false)
const [showCommentModal, setShowCommentModal] = useState<boolean>(false)
const { status } = useSession()
const { data: session, status } = useSession()
const [activeKey, setActiveKey] = useState(CommonTabIds.SUMMARY)

useEffect(() => {
Expand Down Expand Up @@ -467,6 +468,44 @@ const EditRelease = ({ releaseId, isSPDXFeatureEnabled }: Props): ReactNode => {
}
}

const createDuplicate = async () => {
Copy link
Copy Markdown
Contributor

@deo002 deo002 Apr 6, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Duplicate Releases page has only 2 tabs, this one has some unnecessary ones

if (CommonUtils.isNullOrUndefined(session)) {
MessageService.error(t('Session has expired'))
return signOut()
}
try {
const eccInfo = releasePayload.eccInformation
const sanitizedEccInformation: ECCInformation | undefined = eccInfo
? {
...eccInfo,
eccStatus: eccInfo.eccStatus?.trim() !== '' ? eccInfo.eccStatus : undefined,
}
: undefined
const { linkedPackages, clearingState, ...cleanPayload } = releasePayload

const finalPayload: Release = {
...cleanPayload,
eccInformation: sanitizedEccInformation,
}
const response = await ApiUtils.POST('releases', finalPayload, session.user.access_token)

if (response.status === StatusCodes.CREATED) {
const newRelease = (await response.json()) as ReleaseDetail
const newReleaseId = CommonUtils.getIdFromUrl(newRelease._links.self.href)
MessageService.success(`Release ${newRelease.name} (${newRelease.version}) created successfully!`)
router.push('/components/releases/detail/' + newReleaseId)
} else if (response.status === StatusCodes.CONFLICT) {
MessageService.warn(t('Release is Duplicate'))
} else {
const data = await response.json()
MessageService.error(data.message)
}
} catch (e) {
const msg = e instanceof Error ? e.message : String(e)
MessageService.error(msg)
}
}

const checkUpdateEligibility = async (releaseId: string) => {
const session = await getSession()
if (CommonUtils.isNullOrUndefined(session)) return signOut()
Expand Down Expand Up @@ -515,25 +554,46 @@ const EditRelease = ({ releaseId, isSPDXFeatureEnabled }: Props): ReactNode => {
setDeleteModalOpen(true)
}

const headerButtons = {
'Update Release': {
link: '',
type: 'primary',
onClick: checkPreRequisite,
name: t('Update Release'),
},
'Delete Release': {
link: '',
type: 'danger',
onClick: handleDeleteRelease,
name: t('Delete Release'),
},
Cancel: {
link: '/components/releases/detail/' + releaseId,
type: 'secondary',
name: t('Cancel'),
},
}
const headerButtons: {
[key: string]: {
link: string
name: string
type: string
onClick?: () => void
}
} = isDuplicate
? {
'Create Release': {
link: '',
type: 'primary',
onClick: createDuplicate,
name: t('Create Release'),
},
Cancel: {
link: '/components/releases/detail/' + releaseId,
type: 'secondary',
name: t('Cancel'),
},
}
: {
'Update Release': {
link: '',
type: 'primary',
onClick: checkPreRequisite,
name: t('Update Release'),
},
'Delete Release': {
link: '',
type: 'danger',
onClick: handleDeleteRelease,
name: t('Delete Release'),
},
Cancel: {
link: '/components/releases/detail/' + releaseId,
type: 'secondary',
name: t('Cancel'),
},
}

const param = useParams()
const locale = (param.locale as string) || 'en'
Expand Down
Loading