Skip to content

Conversation

@ogabrielides
Copy link

@ogabrielides ogabrielides commented Apr 2, 2023

Issue being fixed or feature implemented

We need to mark a qdata incoming request as processed only when successfully processed. Currently, it is marked upon receive and before actually processing it.

What was done?

How Has This Been Tested?

Breaking Changes

Checklist:

  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have added or updated relevant unit/integration/functional/e2e tests
  • I have made corresponding changes to the documentation

For repository code-owners and collaborators only

  • I have assigned this pull request to a milestone

@ogabrielides ogabrielides marked this pull request as draft April 2, 2023 20:16
@ogabrielides ogabrielides added this to the 19 milestone Apr 2, 2023
@ogabrielides ogabrielides marked this pull request as ready for review April 3, 2023 15:12
@ogabrielides
Copy link
Author

Closed as #5286 is kept instead.

@ogabrielides ogabrielides deleted the qdata_set_process_change branch April 3, 2023 20:49
@UdjinM6 UdjinM6 removed this from the 19 milestone Apr 15, 2023
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