feat: file asset v2 [WPB-16104]

[olafsulich]

WPB-16104 [Web] File assets card

Description

New file asset based on the #18692. The logic mostly remained the same as it was before.

Demo

Uploaded:

Uploading:

Error:

File sharing restricted:

Checklist

• mentions the JIRA issue in the PR name (Ex. [WPB-XXXX])
• PR has been self reviewed by the author;
• Hard-to-understand areas of the code have been commented;
• If it is a core feature, unit tests have been added;

[codecov-commenter]

Codecov Report

Attention: Patch coverage is 2.70270% with 36 lines in your changes missing coverage. Please review.

Project coverage is 45.59%. Comparing base (dfe1d68) to head (c2654a8).
Report is 15 commits behind head on dev.

Additional details and impacted files

@@            Coverage Diff             @@
##              dev   #18749      +/-   ##
==========================================
- Coverage   45.65%   45.59%   -0.06%     
==========================================
  Files         968      974       +6     
  Lines       27937    27977      +40     
  Branches     6302     6312      +10     
==========================================
+ Hits        12754    12757       +3     
- Misses      13594    13628      +34     
- Partials     1589     1592       +3     

[przemvs]

I think You can return directly, because You not reusing this :) And we don't want to support canceled? WE don't display anything if it's canceled ?

Suggested change

	const formattedName = transferNames[transferState as keyof typeof transferNames] ?? name;
	return {
	name: formattedName,
	return {
	name: transferNames[transferState as keyof typeof transferNames] ?? name,

[olafsulich]

I don't think it's necessary - if someone cancels they know what they're doing, and they don't need additional "feedback" on the screen.

Currently, we don't have it, so I'll leave it as it is.

[olafsulich]

And when it comes to having the variable or not, it's a matter of preference I think - let me leave it like that, for more "complex" syntax, I rather have it separated

[przemvs]

Hmm... smth wrong is here.. assetStatus should be AssetTransferState, So You don't have to assert this type as, could You check this ?

[olafsulich]

oh yeah, the condition above was wrong, it's fixed now

[sonarqubecloud]

Quality Gate passed

Issues
5 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud