Skip to content

fix for draft edit link#237

Merged
yashmeet29 merged 1 commit into
developfrom
editLinkDraftFix
Mar 13, 2026
Merged

fix for draft edit link#237
yashmeet29 merged 1 commit into
developfrom
editLinkDraftFix

Conversation

@deepikaSingh2711
Copy link
Copy Markdown
Collaborator

@deepikaSingh2711 deepikaSingh2711 commented Mar 10, 2026

Describe your changes

Problem: After saving an entity with a link attachment, the "Edit Link" button didn't appear when editing the entity again.

Fix: Restore correct mimeType for link attachments after save by checking if linkUrl exists and correcting the mimeType.

Type of change

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

Checklist before requesting a review

  • I have tested the functionality on my cloud environment.
  • I have provided sufficient automated/ unit tests for the code.
  • I have increased or maintained the test coverage.
  • I have ran integration tests on my cloud environment.
  • I have validated blackduck portal for any vulnerability after my commit.

Upload Screenshots/lists of the scenarios tested

  • I have Uploaded Screenshots or added lists of the scenarios tested in description

Single tenant Integration test
https://github.com/cap-js/sdm/actions/runs/22898011199

TENANCY_MODEL=multi TENANT=SDM-DEV-CONSUMER-EU12 npm run integration-test

Screenshot 2026-03-10 at 4 26 23 PM

TENANCY_MODEL=multi TENANT=SDMGoogleWorkspaceConsumer npm run integration-test

Screenshot 2026-03-10 at 4 31 28 PM

@yashmeet29 yashmeet29 merged commit f89fbe0 into develop Mar 13, 2026
8 checks passed
@yashmeet29 yashmeet29 deleted the editLinkDraftFix branch March 13, 2026 05:00
vibhutikumar07 pushed a commit that referenced this pull request Apr 8, 2026
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