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

Vueify log viewer (and use Backstretch) #6738

Merged
merged 14 commits into from
May 29, 2019
Merged

Vueify log viewer (and use Backstretch) #6738

merged 14 commits into from
May 29, 2019

Conversation

sharkykh
Copy link
Contributor

@sharkykh sharkykh commented May 28, 2019

Supersedes #4104

It's basically done!
Looks like I just need to fix the tests.
Edit: Also push the fix for this bug.

  • I added log polling (updates the page every 1.5 seconds), with a button to pause the refreshing.
  • Backstretch component was not being used, fixed it and hopefully got it going with a temporary API variable for a random show slug. It should ultimately be from the list of shows in the store, but they're really available right now.

@sharkykh sharkykh added this to the 0.3.2 milestone May 28, 2019
@OmgImAlexis OmgImAlexis mentioned this pull request May 28, 2019
@ghost

This comment has been minimized.

@ghost

This comment has been minimized.

1 similar comment
@ghost

This comment has been minimized.

@ghost

This comment has been minimized.

1 similar comment
@ghost

This comment has been minimized.

@sharkykh sharkykh added Concluded and removed Changelog Requires a changelog entry Do not merge In progress labels May 29, 2019
@sharkykh

This comment has been minimized.

@p0psicles
Copy link
Contributor

nice PR!

@p0psicles p0psicles merged commit b1a1a86 into develop May 29, 2019
@p0psicles p0psicles deleted the vueify/logs branch May 29, 2019 19:04
@sharkykh
Copy link
Contributor Author

Many thanks

Thilas pushed a commit to Thilas/Medusa that referenced this pull request Jun 5, 2019
* Update backstretch component and use it

* Add `logs.debug`/`logs.dbDebug` to config

* Move dynamic imports in `router.js`

* Convert `/errorlogs/viewlog` to Vue/SFC/API

* Add `thread: "VUE"` option to `POST /api/v2/log`

* Add Logs render test

* Fix logs viewer visual bug

https://i.imgur.com/tP21MW2.jpg

* Fix tests

* Update dredd hook to print to a log file because nothing prints to `stdout`

* Fix log test

* Update test_log.py

* Fix spacing in fixture

* Update CHANGELOG.md

* Should fix `THREAD` != `Thread`
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants