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

Update the import/export documentation #2320

Merged
merged 1 commit into from
Mar 28, 2025

Conversation

lampajr
Copy link
Member

@lampajr lampajr commented Mar 26, 2025

Fixes Issue

Fixes #2263

Changes proposed

  • Improve and fix the existing export/import documentation

Check List (Check all the applicable boxes)

  • My code follows the code style of this project.
  • My change requires changes to the documentation.
  • I have updated the documentation accordingly.
  • All new and existing tests passed.

@lampajr lampajr self-assigned this Mar 26, 2025
@lampajr lampajr added the area/documentation Improvements or additions to documentation label Mar 26, 2025
@lampajr
Copy link
Member Author

lampajr commented Mar 26, 2025

@stalep
Copy link
Member

stalep commented Mar 28, 2025

LGTM, one comment I have is that since we now have eg importSchema/updateSchema methods on the backend, it might be good to have separate UI buttons for import and update as well?

@lampajr
Copy link
Member Author

lampajr commented Mar 28, 2025

LGTM, one comment I have is that since we now have eg importSchema/updateSchema methods on the backend, it might be good to have separate UI buttons for import and update as well?

Yeah I think it could be a good improvement as well, what do you think if I create a separate issue for that?

@stalep
Copy link
Member

stalep commented Mar 28, 2025

LGTM, one comment I have is that since we now have eg importSchema/updateSchema methods on the backend, it might be good to have separate UI buttons for import and update as well?

Yeah I think it could be a good improvement as well, what do you think if I create a separate issue for that?

yes, sounds good!

@stalep stalep merged commit 0c45d58 into Hyperfoil:master Mar 28, 2025
4 checks passed
@lampajr lampajr deleted the test_import_export_doc branch March 28, 2025 13:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Improve documentation on Test import/export
2 participants