Skip to content

Edit link automation#203

Merged
deepikaSingh2711 merged 11 commits intodevelopfrom
editLinkAutomation
Mar 13, 2026
Merged

Edit link automation#203
deepikaSingh2711 merged 11 commits intodevelopfrom
editLinkAutomation

Conversation

@akshatjain-official
Copy link
Collaborator

@akshatjain-official akshatjain-official commented Nov 17, 2025

Describe your changes

Reopened this PR as this was not merged before but was closed

This PR Includes

a) automation test cases for the edit link type of attcahments

  1. Edit an existing link with valid URL
  2. Edit link with unsupported/invalid URL format
  3. Edit link with empty URL
  4. Edit a link without SDM role
  5. Discard draft edited Link

single tenant Integration test
https://github.com/cap-js/sdm/actions/runs/23048712999

multi tenant Integration test
https://github.com/cap-js/sdm/actions/runs/23048231213

b) Updated error message when attempting to edit a link without the SDM role.

Screenshot 2026-03-13 at 4 10 10 PM

c) MIME type validation with an appropriate error message.

image

Any documentation

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.

@yashmeet29 yashmeet29 closed this Feb 25, 2026
@yashmeet29 yashmeet29 reopened this Mar 13, 2026
@deepikaSingh2711 deepikaSingh2711 merged commit 6ba8b90 into develop Mar 13, 2026
8 checks passed
@deepikaSingh2711 deepikaSingh2711 deleted the editLinkAutomation branch March 13, 2026 12:09
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.

3 participants