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

Show all parameters in UI which were passed via -a flag and also have an additional column to "STOP" a particular job #254

Closed
wants to merge 1 commit into from

Conversation

iamumairayub
Copy link

I have modified scrapyD UI to show all parameters which were passed to it, and also have an additional column to "STOP" a particular job ...


s = "<html><head><title>Scrapyd</title>"
s += """
<script src="https://ajax.googleapis.com/ajax/libs/jquery/3.2.1/jquery.min.js"></script>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The scrapyd website is meant to be minimal.
Using javascript libraries if far from this.
This feature can be implemented with plain forms.
We can even change the webservice to allow get requests
and make it as simple as a link next to each job.

@Digenis
Copy link
Member

Digenis commented Nov 8, 2017

Hi @mani619cash,
this is a useful feature that can be included in 1.3.
However I don't agree with the way it's implemented.

Loading jquery from google is too much of a bloat
compared to the current state of scrapyd
which never used javascript.

See my comment in the code,
it can be implemented in a much simpler way.
If you are not sure, I'll prepare a PR.

@Digenis Digenis mentioned this pull request Nov 15, 2017
@jpmckinney
Copy link
Contributor

jpmckinney commented Sep 24, 2021

Closing as abandoned by author. Related open issue: #205

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr: abandoned for unmerged PRs that were abandoned
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants