Skip to content
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

[WEB-522] feat: Estimates revamp #4696

Merged
merged 79 commits into from
Jun 6, 2024

Fix sortable component as the sort order was broken.

53740fc
Select commit
Loading
Failed to load commit list.
Merged

[WEB-522] feat: Estimates revamp #4696

Fix sortable component as the sort order was broken.
53740fc
Select commit
Loading
Failed to load commit list.
Codacy Production / Codacy Static Code Analysis required action Jun 6, 2024 in 0s

4 new issues (0 max.) of at least minor severity.

Annotations

Check notice on line 22 in apiserver/plane/app/views/estimate/base.py

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

apiserver/plane/app/views/estimate/base.py#L22

Depending on the context, generating weak random numbers may expose cryptographic functions, which rely on these numbers, to be exploitable.

Check warning on line 22 in apiserver/plane/app/views/estimate/base.py

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

apiserver/plane/app/views/estimate/base.py#L22

Standard pseudo-random generators are not suitable for security/cryptographic purposes.

Check notice on line 37 in apiserver/plane/db/migrations/0067_issue_estimate.py

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

apiserver/plane/db/migrations/0067_issue_estimate.py#L37

Trailing whitespace

Check notice on line 84 in web/components/estimates/root.tsx

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

web/components/estimates/root.tsx#L84

Using target="_blank" without rel="noreferrer" (which implies rel="noopener") is a security risk in older browsers: see https://mathiasbynens.github.io/rel-noopener/#recommendations