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

chore(sdf, dal): Cleanup old pre-REBAC routes #5247

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

stack72
Copy link
Contributor

@stack72 stack72 commented Jan 14, 2025

No description provided.

@github-actions github-actions bot added A-sdf Area: Primary backend API service [Rust] A-dal A-web labels Jan 14, 2025
Copy link

github-actions bot commented Jan 14, 2025

Dependency Review

✅ No vulnerabilities or OpenSSF Scorecard issues found.

OpenSSF Scorecard

PackageVersionScoreDetails

Scanned Files

path: () => "/change_set/begin_abandon_approval_process",
force_apply: {
path: (vars: ROUTE_VARS) =>
`/v2/workspaces/${vars.workspaceId}/change-sets/${vars.changeSetId}/force_apply`,
method: "POST",
},
begin_approval_process: {
Copy link
Contributor

Choose a reason for hiding this comment

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

are we using this still? or can this one get cleaned up?

Way back when, I was considering keeping the low-fi approval flow for things like 'abandon change set' (as it gives people who are on the change set a chance to stop the abandon if they're actively doing something) but wasn't totally sure...

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I do see that an endpoint still is called so I left it in place - it's when the user is an approver or an owner and they force their own approval

Copy link
Contributor

Choose a reason for hiding this comment

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

Sorry bad github comment location - I meant the begin_approval_process pieces

Copy link
Contributor

Choose a reason for hiding this comment

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

Definitely not for this PR - just leaving a note that we need to update both this client and the tests to account for views when we clean up that feature flag

@stack72 stack72 force-pushed the chore-cleanup-sdf-post-rebac branch 2 times, most recently from 4db4e79 to 3d907e6 Compare January 15, 2025 23:54
@britmyerss britmyerss self-requested a review January 16, 2025 00:23
britmyerss
britmyerss previously approved these changes Jan 16, 2025
@stack72
Copy link
Contributor Author

stack72 commented Jan 18, 2025

Ok, so I was able to test this by going through the flow of merging a change set in a workspace as the owner. Then going through the request approval, reject, make edits, request approval and approve

Screenshot 2025-01-18 at 15 59 50
Screenshot 2025-01-18 at 16 00 26
Screenshot 2025-01-18 at 16 00 39
Screenshot 2025-01-18 at 16 00 45
Screenshot 2025-01-18 at 16 00 49
Screenshot 2025-01-18 at 16 00 55
Screenshot 2025-01-18 at 16 01 06

@stack72
Copy link
Contributor Author

stack72 commented Jan 18, 2025

API Tests running against this branch

~~ FINAL REPORT GENERATED ~~
Test Report:
[
  {
    "test_name": "6-get_head_changeset",
    "start_time": "2025-01-18T17:12:55.298Z",
    "finish_time": "2025-01-18T17:12:55.352Z",
    "test_duration": "54ms",
    "test_result": "success",
    "message": "",
    "test_execution_sequence": 7,
    "uuid": "5cf2f54e-e297-4e05-8a3f-eec999403e1c"
  },
  {
    "test_name": "0-add_action",
    "start_time": "2025-01-18T17:12:55.295Z",
    "finish_time": "2025-01-18T17:12:56.028Z",
    "test_duration": "733ms",
    "test_result": "success",
    "message": "",
    "test_execution_sequence": 1,
    "uuid": "fbd70edd-2cc0-4c21-864b-c8a7a60669dd"
  },
  {
    "test_name": "2-create_and_delete_component",
    "start_time": "2025-01-18T17:12:55.298Z",
    "finish_time": "2025-01-18T17:12:56.816Z",
    "test_duration": "1518ms",
    "test_result": "success",
    "message": "",
    "test_execution_sequence": 5,
    "uuid": "23ec7375-fab3-48ea-8fc6-6d98a7b8bb38"
  },
  {
    "test_name": "3-create_and_use_variant",
    "start_time": "2025-01-18T17:12:55.297Z",
    "finish_time": "2025-01-18T17:12:57.761Z",
    "test_duration": "2464ms",
    "test_result": "success",
    "message": "",
    "test_execution_sequence": 3,
    "uuid": "2f2846c3-0791-4a2f-8dff-cced6d23d78b"
  },
  {
    "test_name": "4-create_two_components_connect_and_propagate",
    "start_time": "2025-01-18T17:12:55.298Z",
    "finish_time": "2025-01-18T17:13:05.403Z",
    "test_duration": "10105ms",
    "test_result": "success",
    "message": "",
    "test_execution_sequence": 6,
    "uuid": "4d621ff0-6fda-4484-82e8-f311f44e9219"
  },
  {
    "test_name": "7-emulate_authoring",
    "start_time": "2025-01-18T17:12:55.297Z",
    "finish_time": "2025-01-18T17:13:08.013Z",
    "test_duration": "12716ms",
    "test_result": "success",
    "message": "",
    "test_execution_sequence": 2,
    "uuid": "60eef501-a1ae-41e1-80f9-091dae3a17ad"
  },
  {
    "test_name": "5-emulate_paul_stack",
    "start_time": "2025-01-18T17:12:55.298Z",
    "finish_time": "2025-01-18T17:13:16.364Z",
    "test_duration": "21066ms",
    "test_result": "success",
    "message": "",
    "test_execution_sequence": 8,
    "uuid": "1b45e531-b0ac-44f8-8e34-e04f03bf45ba"
  },
  {
    "test_name": "1-create_and_apply_across_change_sets",
    "start_time": "2025-01-18T17:12:55.297Z",
    "finish_time": "2025-01-18T17:13:17.871Z",
    "test_duration": "22574ms",
    "test_result": "success",
    "message": "",
    "test_execution_sequence": 4,
    "uuid": "89ed0246-c643-4cfb-801b-bc19d987cd60"
  }
]

@stack72 stack72 requested a review from britmyerss January 18, 2025 19:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-dal A-sdf Area: Primary backend API service [Rust] A-web
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants