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

refactor: convert properties modal test to rtl #18241

Conversation

em0227
Copy link
Contributor

@em0227 em0227 commented Feb 1, 2022

SUMMARY

in progress

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TESTING INSTRUCTIONS

ADDITIONAL INFORMATION

  • Has associated issue:
  • Required feature flags:
  • Changes UI
  • Includes DB Migration (follow approval process in SIP-59)
    • Migration is atomic, supports rollback & is backwards-compatible
    • Confirm DB migration upgrade and downgrade tested
    • Runtime estimates and downtime expectations provided
  • Introduces new feature or API
  • Removes existing feature or API

@superset-github-bot superset-github-bot bot added the Superset-Community-Partners Preset community partner program participants label Feb 1, 2022
@em0227 em0227 changed the title refactor:convert properties modal test to rtl refactor: convert properties modal test to rtl Feb 1, 2022
@stale
Copy link

stale bot commented Apr 16, 2022

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. For admin, please label this issue .pinned to prevent stale bot from closing the issue.

@stale stale bot added the inactive Inactive for >= 30 days label Apr 16, 2022
@stale stale bot removed the inactive Inactive for >= 30 days label Apr 18, 2022
@rusackas
Copy link
Member

@em0227 I certainly applaud the efforts to modernize these tests! Are you intending to follow up on this, or shall we go ahead and close the PR?

1 similar comment
@rusackas
Copy link
Member

@em0227 I certainly applaud the efforts to modernize these tests! Are you intending to follow up on this, or shall we go ahead and close the PR?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size/L Superset-Community-Partners Preset community partner program participants
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants