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

[MDS-5833] allow reissuance after revocation #3025

Merged
merged 22 commits into from
Mar 26, 2024

Conversation

Jsyro
Copy link
Collaborator

@Jsyro Jsyro commented Mar 25, 2024

Objective

MDS-5833

Active and valid
image

Immediately after revoked (not available for re-issuance)
image

Revoked but made available to second issuance
image

Other Changes:
Objects modified by webhooks have a timestamp of the webhooks. this will mean if packets are received in the wrong order a newer state will not be overriden by an older state because the packets arrived in the reverse order.

@Jsyro Jsyro requested a review from matbusby-fw March 25, 2024 22:58
@Jsyro Jsyro marked this pull request as ready for review March 26, 2024 16:22
@Jsyro Jsyro requested a review from taraepp March 26, 2024 16:31
@@ -4,6 +4,9 @@ import CustomAxios from "@mds/common/redux/customAxios";
import { ENVIRONMENT, MINES_ACT_PERMITS_VC_LIST } from "@mds/common/constants";
import * as API from "@mds/common/constants/API";
import { RootState } from "@mds/common/redux/rootState";
import { DataSourceItemType } from "antd/lib/auto-complete";
import { debug } from "webpack";
import { a } from "@mds/common/tests/mocks/dataMocks";

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What is 'a'? And is this + debug something that's no longer being used? I would guess it might cause issues with variables on sort(a,b)

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

removed

Copy link

Quality Gate Passed Quality Gate passed for 'bcgov-sonarcloud_mds_minespace-web'

Issues
1 New issue
0 Accepted issues

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

See analysis details on SonarCloud

Copy link

Quality Gate Failed Quality Gate failed for 'bcgov-sonarcloud_mds_common'

Failed conditions
0.0% Coverage on New Code (required ≥ 80%)

See analysis details on SonarCloud

Copy link

Quality Gate Failed Quality Gate failed for 'bcgov-sonarcloud_mds_core-api'

Failed conditions
0.0% Coverage on New Code (required ≥ 80%)

See analysis details on SonarCloud

Copy link

Quality Gate Failed Quality Gate failed for 'bcgov-sonarcloud_mds_core-web'

Failed conditions
0.0% Coverage on New Code (required ≥ 80%)

See analysis details on SonarCloud

@Jsyro Jsyro merged commit db25422 into develop Mar 26, 2024
19 of 22 checks passed
@Jsyro Jsyro deleted the feature/5833-allow-reissuance branch March 26, 2024 19:35
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