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

change structure of squish setup #68

Merged
merged 1 commit into from
Jun 25, 2021
Merged

change structure of squish setup #68

merged 1 commit into from
Jun 25, 2021

Conversation

swoichha
Copy link
Contributor

@swoichha swoichha commented Jun 25, 2021

Description

After #62 has been merged there are some minor changes that needed to be done in the documentation for GUI Testing the Desktop Client. Following are the changes made:

  • Move Install Squish and Configure Squish from Prerequisites to inside of Prepare Tests -> Using Squish IDE
  • remove Installing Squish Using an Executable File heading
  • python changer script did not have an URL but pythonchanger-url[Download]
  • replace license key with license/url of the license server to make it more clearer.

@swoichha swoichha self-assigned this Jun 25, 2021
@swoichha swoichha added the documentation Improvements or additions to documentation label Jun 25, 2021
Copy link
Contributor

@EParzefall EParzefall left a comment

Choose a reason for hiding this comment

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

looks good

@mmattel
Copy link
Contributor

mmattel commented Jun 25, 2021

Found and fixed a broken internal link because of a typo. Squashed and Pushed

@mmattel mmattel merged commit 78b1ed5 into master Jun 25, 2021
@delete-merged-branch delete-merged-branch bot deleted the docs-gui-tests branch June 25, 2021 14:26
mmattel added a commit that referenced this pull request Jun 25, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants