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

Macro: #3041 – RNA Builder. Delete presets. #3414

Merged
merged 19 commits into from
Oct 18, 2023

Conversation

StarlaStarla
Copy link
Contributor

@StarlaStarla StarlaStarla commented Oct 9, 2023

How the feature works? / How did you fix the issue?

(Screenshots, videos, or GIFs, if applicable)
image
image
image
image

Check list

  • unit-tests written
  • e2e-tests written
  • documentation updated
  • PR name follows the pattern #1234 – issue name
  • branch name doesn't contain '#'
  • PR is linked with the issue
  • base branch (master or release/xx) is correct
  • task status changed to "Code review"
  • reviewers are notified about the pull request

@StarlaStarla StarlaStarla linked an issue Oct 9, 2023 that may be closed by this pull request
@StarlaStarla StarlaStarla changed the title #3041 Delete RNA presets. #3041- RNA Builder. Delete presets. Oct 9, 2023
await takePageScreenshot(page);
});

test.skip('Delete copy RNA preset', async ({ page }) => {
Copy link
Collaborator

Choose a reason for hiding this comment

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

Why did you skip it?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Because I saw this in the README.md
image

Copy link
Collaborator

Choose a reason for hiding this comment

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

@rrodionov91 Is it up-to-date information?

Copy link
Collaborator

Choose a reason for hiding this comment

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

@OlgaMazurina , @StarlaStarla it is old information. Currently tests are running on CI also, so we don't need to skip it. Please also remove this info from readme

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

await waitForPageInit(page);
await gotoRNA(page);
});

Copy link
Collaborator

Choose a reason for hiding this comment

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

Let's add here test.afterEach:
await test.afterEach(async ({ page }) => {
await takePageScreenshot(page);
});

@StarlaStarla StarlaStarla force-pushed the 3041-rna-builder-delete-presets branch from f3599cd to e243fdb Compare October 10, 2023 04:40
@StarlaStarla StarlaStarla force-pushed the 3041-rna-builder-delete-presets branch from 537c5ec to 2711c70 Compare October 12, 2023 03:16
@Nitvex Nitvex changed the title #3041- RNA Builder. Delete presets. Macro: #3041 – RNA Builder. Delete presets. Oct 13, 2023
@rrodionov91 rrodionov91 merged commit b07a313 into master Oct 18, 2023
5 checks passed
@rrodionov91 rrodionov91 deleted the 3041-rna-builder-delete-presets branch October 18, 2023 12:55
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.

Macro: RNA Builder. Delete presets.
6 participants