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
Issues/ 92 Add PR Reminder Panel #100
Conversation
@@ -7,5 +7,6 @@ export default { | |||
presentations: `http://${dashHost}:8083`, | |||
meetings: `http://${dashHost}:8084`, | |||
infrastructure: `http://${dashHost}:8085`, | |||
github: `http://${dashHost}:8086` | |||
github: `http://${dashHost}:8086`, | |||
reminder: `http://${dashHost}:8086/pull-requests` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The dashboard no longer uses this config file (we should probably remove it). It will instead automagically fetch from the proper container in the Docker network based on the panel's name (i.e. http://data/{container-name}
). I believe the getData
function in Service.js
will need to be changed to support fetching from a specific path
(e.g. /pull-requests
), since we would previously only fetch from the root
endpoint of each container.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's make removing Dashboard/src/config.js
part of a separate issue.
69ba7b4
to
6e389e5
Compare
Closes #92
The PR Reminder panel shows the following info (3 lines per PR):
number
,PR title
,repo
,date
,reviewers
<--(icons, not text)labels
description
(truncated, shows complete description when hovering over)Also,
dummy-data.js
now has PR Reminder data for testing.The reminders don't include a
url
to the PR, issue or branch because the back-end doesn't provide that information. We should probably open an issue for that.