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

Support uploading log.html #5

Open
adiralashiva8 opened this issue Feb 18, 2020 · 2 comments
Open

Support uploading log.html #5

adiralashiva8 opened this issue Feb 18, 2020 · 2 comments
Assignees
Labels
enhancement New feature or request

Comments

@adiralashiva8
Copy link
Owner

As a rfhistoric user I want to store my log.html into DB so that I can download when required in future

@adiralashiva8 adiralashiva8 added the enhancement New feature or request label Feb 18, 2020
@adiralashiva8 adiralashiva8 self-assigned this Feb 18, 2020
@adiralashiva8 adiralashiva8 added the help wanted Extra attention is needed label Feb 19, 2020
@adiralashiva8 adiralashiva8 removed the help wanted Extra attention is needed label Mar 6, 2020
@SiNeumann
Copy link

@adiralashiva8 Any plans for that? I might give it a shot and work on PR for this in the upcoming weeks/month (depends on my time).
Any considerations for this feature?
Would you parse the log.html and save it in chuncks (ie. per Tests) so it can be linked. per case. Or just dump the log.html in the db?

@adiralashiva8
Copy link
Owner Author

Hi @SiNeumann Thanks for showing interest on feature. I like to store log.html (text) and provide link to download it on demand

My expectations:

  • New page in respective project
  • Page should accept execution id and log.html. (following is my plan on log.html)
    • Support drag and drop html file (bit complex and may not support remote access)
    • Ask user to copy .html content (this should be fine, until it support feature)
  • New link to download above uploaded content for respective execution id (historic tab)

I'm really excited to have this feature but we don't have extra columns in dB. New feature implementation doesn't support existing projects so I dropped from implementation

Please have a try. Thank you

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants