Update QA learning goals and Session Topics #246
Conversation
adamblanchard
left a comment
adamblanchard
left a comment
There was a problem hiding this comment.
Thanks @GayathriVenkatraman! just a couple of small comments.
I'm a little confused why my commits/changes are still showing up in here when I merged the other PR. Can you update your branch with the latest main and see if that helps? Otherwise let me know if you can see why, I'm a bit stumped, so must be missing something 🤯 😁
| @@ -1,0 +1,3 @@ | |||
| { | |||
There was a problem hiding this comment.
Would you mind removing this? cSpell isn't currently an official plugin in this project so might be better to keep this in your user settings rather than committing it for others too :)
| "location": "courses/qa", | ||
| "modules": [ | ||
| { | ||
| "name": "Basics of Testing", |
There was a problem hiding this comment.
Can you update the "name" values in this file too, based on the new names you wrote for the above modules? 🙇 This json is used to autogenerate different docs in the program, so we should replace them too.
2 participants
@adamblanchard I have updated the "Learning Goals" for Final Project, also I have reviewed your PR (#242) . We could merge and close the PR 242 (as I do not have the option to merge the PR).
This PR has now updated "Learning Goals", also I made some changes on the module title and the session title.
@adamblanchard Please take a look and let's merge this PR as well.