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

Resolved issue #178 (switch from GET->POST if payload exists) #513

Closed
wants to merge 2 commits into from

Conversation

AB527
Copy link

@AB527 AB527 commented Dec 19, 2024

PR Description

This PR is with respect to the issue #178 . I have resolved this issue and also added the relevant test cases.

The final results are as follows:

Screen.Recording.2024-12-19.005210.mp4

And the test case results:

image

Related Issues

Checklist

  • [✅] I have gone through the contributing guide
  • [✅] I have updated my branch and synced it with project main branch before making this PR
  • [✅] I have run the tests (flutter test) and all tests are passing

Added/updated tests?

  • [✅] Yes
  • No, and this is why: please replace this line with details on why tests have not been included

OS on which you have developed and tested the feature?

  • [✅] Windows
  • macOS
  • Linux

@AB527 AB527 changed the title Resolve issue 178 Resolved issue #178 (switch from GET->POST if payload exists) Dec 19, 2024
@ashitaprasad
Copy link
Member

@AB527 You have reverted various changes that were recently made to the project.
Kindly, take a fresh fork of the project or update your fork. Then make the requisite changes and raise a new PR.

@AB527
Copy link
Author

AB527 commented Dec 19, 2024

@ashitaprasad I have updated my fork and raised a new PR.

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.

2 participants