Skip to content

83631: Provide support for catch the returned response body in uploader component #2902

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

Merged
merged 5 commits into from
Apr 26, 2024

Conversation

sureshsf3950
Copy link
Contributor

@sureshsf3950 sureshsf3950 commented Apr 22, 2024

Feature description

Provide support for catch the returned response body in uploader component.

Requirement and specification document.

API Review task

Output screenshots

image

Feature matrix document

Feature matrix document updated against this feature and committed in this [common location] (https://syncfusion.sharepoint.com/sites/EJ2ProductOwners/Shared%20Documents/Forms/AllItems.aspx?viewid=ae81c682%2D3d0f%2D462a%2Db8ec%2D7358748d489d&id=%2Fsites%2FEJ2ProductOwners%2FShared%20Documents%2FGeneral%2FFeature%20Matrix%20%2D%20Documents) .

  • Yes
  • NO
  • NA

Provide the details about the areas or combinations which have been tested against this code changes.

  • Tested against feature matrix.

Feature acceptance criteria (Test-case document)

Draft the test cases in excel and attach it in the MR itself. The automation must be covered based on this positive and worst-case test-cases.

Automation details

Mark 'Is Automated' field as (Yes, Manual, Not Applicable) in corresponding JIRA task once the feature is automated.

  • BUnit, share corresponding MR.
  • E2E or Manual Automation using tester - Make sure all items are automated with priority before release which can be tracked in automation dashboard.

Test bed sample location

Provide the test bed sample location where the code reviewers can review the new feature’s behaviors.

Feature completion checklist

UX changes got approval if UI is added or modified?

  • Yes. Provide Task link.
  • NO
  • NA

Drafted UG for this feature?

  • Yes, provide MR link.
  • NO, provide task link.
  • NA

Content team reviewed the UI content changes.

  • Yes. Provide MR or Task link.
  • NO
  • NA

Is this the common feature which needs to be addressed in the same component or on other components in our platform?

  • Yes. If we need to check in other components, tag "needs-attention-coreteam"
  • NO

Is there any existing behavior change due to this code change?

  • Yes. If So, add breaking-change label.
  • NO

Blazor Checklist

Confirm whether the ensured feature is in both Blazor Server and WASM.

  • Yes
  • NO
  • NA

Do the code changes cause any memory leak and performance issue?

  • Yes
  • NO

Reviewer Checklist

  • Reviewed feature matrix document modifications and reviewed developer testing report against the feature matrix reviewed.
  • Testbed sample ensured.
  • Coverage report checked.
  • Ensured the code changes meet the coding standard.
  • Confirming script changes made in this merge request can't be achieved in C#.

Created task in Blazor to synchronize if the feature is not available and applicable.

  • NA
  • Yes, task link.
  • NO, share reason.

@Syncfusion-CI
Copy link

Build Status: INQUEUE 🕒
Build Location: 83631-dev

@Syncfusion-CI
Copy link

Build Status: INPROGRESS 🔃
Build Location: 83631-dev

@Syncfusion-CI
Copy link

The running CI Job is terminated due to changes committed on the source branch for this Merge Request and CI triggered for latest commit.

@Syncfusion-CI
Copy link

CI Status: ABORTED ❌
Source Compilation: Not Started ❌
Build Location: 83631-dev

@Syncfusion-CI
Copy link

Build Status: INQUEUE 🕒
Build Location: 83631-dev

@Syncfusion-CI
Copy link

Build Status: INPROGRESS 🔃
Build Location: 83631-dev

@SyncfusionBuild
Copy link
Collaborator

CI Status: FAILURE ❌
Source Compilation: SUCCESS ✅
Failure Reason: CI Failure with violation
Build Location: 83631-dev
Technical Error(s): 0
Spelling Error(s): 4
Image Alt Text Error(s): 0
Image Size Error(s): 0
Image Name Error(s): 0
FT Layout Syntax Error(s): 0
Front matter Error(s): 0
File Path Error(s): 0
CodeSnippet File Path Missing Error(s): 0
Toc Anchor Tags Validation Error(s): 0

@Syncfusion-CI
Copy link

Build Status: INQUEUE 🕒
Build Location: 83631-dev

@SyncfusionBuild
Copy link
Collaborator

Build Status: INPROGRESS 🔃
Build Location: 83631-dev

@SyncfusionBuild
Copy link
Collaborator

CI Status: SUCCESS ✅
Source Compilation: SUCCESS ✅
Build Location: 83631-dev
Technical Error(s): 0
Spelling Error(s): 0
Image Alt Text Error(s): 0
Image Size Error(s): 0
Image Name Error(s): 0
FT Layout Syntax Error(s): 0
Front matter Error(s): 0
File Path Error(s): 0
CodeSnippet File Path Missing Error(s): 0
Toc Anchor Tags Validation Error(s): 0

@sureshsf3950 sureshsf3950 changed the title 83631: Provide support for catch the returned response body in upload 83631: Provide support for catch the returned response body in uploader component Apr 23, 2024
@SyncfusionBuild
Copy link
Collaborator

Build Status: INPROGRESS 🔃
Build Location: 83631-dev

@SyncfusionBuild
Copy link
Collaborator

CI Status: FAILURE ❌
Source Compilation: SUCCESS ✅
Failure Reason: CI Failure with violation
Build Location: 83631-dev
Technical Error(s): 0
Spelling Error(s): 2
Image Alt Text Error(s): 0
Image Size Error(s): 0
Image Name Error(s): 0
FT Layout Syntax Error(s): 0
Front matter Error(s): 0
File Path Error(s): 0
CodeSnippet File Path Missing Error(s): 0
Toc Anchor Tags Validation Error(s): 0

@SyncfusionBuild
Copy link
Collaborator

Build Status: INQUEUE 🕒
Build Location: 83631-dev

@SyncfusionBuild
Copy link
Collaborator

Build Status: INPROGRESS 🔃
Build Location: 83631-dev

@SyncfusionBuild
Copy link
Collaborator

CI Status: SUCCESS ✅
Source Compilation: SUCCESS ✅
Build Location: 83631-dev
Technical Error(s): 0
Spelling Error(s): 0
Image Alt Text Error(s): 0
Image Size Error(s): 0
Image Name Error(s): 0
FT Layout Syntax Error(s): 0
Front matter Error(s): 0
File Path Error(s): 0
CodeSnippet File Path Missing Error(s): 0
Toc Anchor Tags Validation Error(s): 0

@PrinceOliver PrinceOliver merged commit b99ae59 into development Apr 26, 2024
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.

4 participants