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

[UI] Support triggering tasks in response to pulse messages #221

Closed
helfi92 opened this issue Jan 3, 2019 · 7 comments
Closed

[UI] Support triggering tasks in response to pulse messages #221

helfi92 opened this issue Jan 3, 2019 · 7 comments
Labels
ui tc-web front end

Comments

@helfi92
Copy link
Contributor

helfi92 commented Jan 3, 2019

This is similar as taskcluster/taskcluster-tools#595 but for taskcluster-web.

@helfi92
Copy link
Contributor Author

helfi92 commented Jan 4, 2019

👋 @ydidwania

A couple of tips for the new UI:

  • The HookForm component has the hook form.
  • We use material-ui for our components
  • We use jss-in-css for styling. See here for more information.
  • taskcluster-web uses taskcluster-web-server for its backend. It's basically graphql sitting on top of our services. This helps speed up performance in the UI because it lets the client request only the information it needs unline REST where it gives you everything an endpoint has. Feel free to read [1] and [2] to learn a bit more about graphql and why we chose to do this.

[1] https://hassanali.me/2018/05/18/rethinking-our-ui.html
[2] https://hassanali.me/2018/05/18/server-side-graphql.html

I can't wait to see what you come up with. Let me know if you have any questions :)

@helfi92
Copy link
Contributor Author

helfi92 commented Jan 14, 2019

Hey @ydidwania, how is going? Let me know if you have any questions :)

@ydidwania
Copy link
Collaborator

ydidwania commented Jan 15, 2019

@helfi92 I did start with it. I got stuck somewhere in trying to add bindings to hook definitions at multiple places in tc-web and tc-web-server. Let me try again

@helfi92
Copy link
Contributor Author

helfi92 commented Jan 15, 2019

Sounds good, many thanks for the update. Let me know if you had any questions for me.

@helfi92 helfi92 transferred this issue from taskcluster/taskcluster-web Feb 13, 2019
@helfi92 helfi92 changed the title Support triggering tasks in response to pulse messages [UI] Support triggering tasks in response to pulse messages Feb 13, 2019
@helfi92 helfi92 added the ui tc-web front end label Feb 13, 2019
@djmitche
Copy link
Collaborator

Looks like this needs to be finished.. @ydidwania do you have time to work on it?

@ydidwania
Copy link
Collaborator

@djmitche I am sorry I cannot work on this for now. I can maybe get back to it around end of July. Is that okay?

@djmitche
Copy link
Collaborator

djmitche commented Jun 5, 2019

Ah, now that I look more closely @arshadkazmi42 is working on this in #730. So, all good :)

@djmitche djmitche closed this as completed Jun 5, 2019
petemoore added a commit that referenced this issue Feb 6, 2020
Bug 1565215 - include timezone in log timestamps and set to UTC
imbstack pushed a commit that referenced this issue May 5, 2020
Change wording saying image was loaded
imbstack pushed a commit that referenced this issue May 7, 2020
Change wording saying image was loaded
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ui tc-web front end
Projects
None yet
Development

No branches or pull requests

3 participants