zend-developer-tools icon indicating copy to clipboard operation
zend-developer-tools copied to clipboard

Page for background-collection

Open Blaimi opened this issue 9 years ago • 5 comments

this PR sets up a page, where you can view former collected toolbar entries by accessing the route /zdt-background-reque.

Known Bugs:

  • It’s ugly
  • If you scroll to the page’s bottom, it’s horrible to read the bar’s content
  • it uses a very high amount of memory

Missing:

  • Tests
  • Some cookie-stuff to assign and filter the requests (that’s, what the :uuid in the route is for)
  • A way to cleanup the cache
  • Review

Blaimi avatar May 03 '16 15:05 Blaimi

@Blaimi Are you still working on this? If so, could you rebase, and also please provide a screenshot detailing what is displayed? I'm not entirely sure what this will do in its current state. :smile:

Thanks in advance!

weierophinney avatar Sep 08 '16 13:09 weierophinney

We are using this version in our development-process but for zend 2.4.

It’s working, but not very intuitive, very slow and it crashes, if you have more than ~25 lines/toolbars. Since the cache is not cleaning up itself and the page displays everytime everything available 25 lines are reached very fast.

To answer your question: yes, we are still working with it. We are planning to migrate/update to Zend3 but not in the next few weeks. Maybe in Q1/17 along with the migration to Debian 9 and php7. I don’t think that we have much spare time to invest in this project since we are unable to use the current version of zdt (depends on zend ^3.0) in our own development-process.

zdt-background-requests

Blaimi avatar Sep 09 '16 12:09 Blaimi

I think this is definitely an interesting feature. However, we need unit tests for the new functionality.

weierophinney avatar Apr 17 '18 15:04 weierophinney

This repository has been closed and moved to laminas/laminas-developer-tools; a new issue has been opened at https://github.com/laminas/laminas-developer-tools/issues/5.

weierophinney avatar Dec 31 '19 22:12 weierophinney

This repository has been moved to laminas/laminas-developer-tools. If you feel that this patch is still relevant, please re-open against that repository, and reference this issue. To re-open, we suggest the following workflow:

  • Squash all commits in your branch (git rebase -i origin/{branch})
  • Make a note of all changed files (`git diff --name-only origin/{branch}...HEAD
  • Run the laminas/laminas-migration tool on the code.
  • Clone laminas/laminas-developer-tools to another directory.
  • Copy the files from the second bullet point to the clone of laminas/laminas-developer-tools.
  • In your clone of laminas/laminas-developer-tools, commit the files, push to your fork, and open the new PR. We will be providing tooling via laminas/laminas-migration soon to help automate the process.

weierophinney avatar Dec 31 '19 22:12 weierophinney