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

fix: [DHIS2-18278] remove completedAt and completedBy from the payload send to the API #3873

Merged
merged 2 commits into from
Nov 5, 2024

Conversation

simonadomnisoru
Copy link
Contributor

@simonadomnisoru simonadomnisoru commented Nov 5, 2024

DHIS2-18278

Tech summary:

  • remove completedAt and completedBy from the payload sent to the API
  • fix the cypress tests

@simonadomnisoru simonadomnisoru marked this pull request as ready for review November 5, 2024 08:33
@simonadomnisoru simonadomnisoru requested a review from a team as a code owner November 5, 2024 08:33
Copy link
Contributor

@eirikhaugstulen eirikhaugstulen left a comment

Choose a reason for hiding this comment

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

LGTM!

Copy link

github-actions bot commented Nov 5, 2024

Copy link

@geethaalwan geethaalwan left a comment

Choose a reason for hiding this comment

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

Tested successfully on 2.42. version

@simonadomnisoru simonadomnisoru merged commit 156369a into master Nov 5, 2024
63 checks passed
@simonadomnisoru simonadomnisoru deleted the DHIS2-18278 branch November 5, 2024 09:35
dhis2-bot added a commit that referenced this pull request Nov 5, 2024
## [101.14.8](v101.14.7...v101.14.8) (2024-11-05)

### Bug Fixes

* [DHIS2-18278] remove completedAt and completedBy from the payload send to the API ([#3873](#3873)) ([156369a](156369a))
@dhis2-bot
Copy link
Contributor

🎉 This PR is included in version 101.14.8 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants