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

Restore jobs scheduling interface and capabilities #728

Merged
merged 1 commit into from
Jun 29, 2017
Merged

Conversation

flovilmart
Copy link
Contributor

@flovilmart flovilmart commented Jun 14, 2017

This goes side by side with parse-community/parse-server#3927

This restores the Scheduled Jobs section, keeps the All Jobs to run them manually, modernizes the network calls by using Parse._request instead of AJAX.

@flovilmart
Copy link
Contributor Author

@natanrolnik what do you think?

@natanrolnik
Copy link
Contributor

In general I think it looks good. I just think we need to make it super clear to users that it's the user's responsibility to actually trigger the jobs, differently than what it used to be with Parse.com and how people would expect it to work out of the box.
We should maybe add a message in the same area as the jobs scheduling and add it to the server docs?

@flovilmart
Copy link
Contributor Author

flovilmart commented Jun 19, 2017

Yes that's a great idea! I'll put something in the Scheduled section

@natanrolnik
Copy link
Contributor

@flovilmart waiting for your change to merge this one, as parse-community/parse-server#3927 was already merged

@flovilmart
Copy link
Contributor Author

Yeppy yay!

@flovilmart
Copy link
Contributor Author

@natanrolnik actually, I kinda struggle setting that up, me no good with React!\

@natanrolnik
Copy link
Contributor

I’ll try to do it in the next few days then.

@natanrolnik
Copy link
Contributor

Let's get this in, @ssamuli will help us later on adding the message.

@natanrolnik natanrolnik merged commit 9227872 into master Jun 29, 2017
@dplewis dplewis deleted the jobs-support branch May 9, 2019 21:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants