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 sdk records test case in record mode #63

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

tanmay-bajaj
Copy link

Fixes: #430
TS-SDK was saving tests in test mode also which shouldn't be the case, it should only save tests in the record mode.
I have figured out the correct location to apply the correct if condition.

@developer-diganta
Copy link

developer-diganta commented Mar 10, 2023

@re-Tick Can you please review this? @Frey0-0 Please wait while the maintainers review this. After that you can add this in the completed task list in the gsoc repo!

@tanmay-bajaj
Copy link
Author

Sure @developer-diganta, can you point me to some other issues in the meantime? Would be really kind of you 🚀

@tanmay-bajaj
Copy link
Author

Are there any more changes required on this PR or should I add this in the completed tasks list? @re-Tick

@re-Tick
Copy link
Contributor

re-Tick commented Mar 12, 2023

@Frey0-0 please add link to this PR in GSOC/2023 ts-sdk task list corresponding to its issue.

@tanmay-bajaj
Copy link
Author

Done!

@developer-diganta developer-diganta added Don't Merge Not to be merged until gsoc results Accepted PR is reviewed and Accepted labels Mar 13, 2023
@developer-diganta
Copy link

Sure @developer-diganta, can you point me to some other issues in the meantime? Would be really kind of you 🚀

Hey @Frey0-0 ! Congrats on completing this PR! Also, you can try out the other tasks in the task list for typescript sdk or any other project that you would want to contribute to!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Accepted PR is reviewed and Accepted Don't Merge Not to be merged until gsoc results
Projects
None yet
3 participants