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

Add Pulls and Pushes view to individual repo #326

Open
wants to merge 2 commits into
base: vue
Choose a base branch
from

Conversation

NicsWorld
Copy link

@NicsWorld NicsWorld commented Mar 12, 2020

This PR adds a filtering of the Activity feed by pull requests and pushes to each individual repo view.

Some questions I wanted to clarify -

  1. There were existing lint errors, I cleaned up a few that I saw but there are still some, should I clean all of them up?
  2. The readme calls out running tests, but I could not find any. Am I missing them?

Example:

drone-ui

@bradrydzewski
Copy link
Member

bradrydzewski commented Mar 13, 2020

Hey @NicsWorld we should touch base in Gitter to discuss.

We have some code in the vue-1.0.0 branch for dedicated branch and deployment screens. You can see a screenshot here:

Screen Shot 2020-03-13 at 6 20 43 PM

The above interaction is more comparable to the GitHub branches and pull request pages, showing the latest status for each:

Screen Shot 2020-03-13 at 6 31 08 PM

This is something we discussed quite a bit when this interaction was designed. We felt that showing the latest build per branch is the optimal user interaction most of the time, however, there will definitely be times when you want to drill down to see a list of recently builds for a specific branch. It would be great to augment this interaction with a drill-down. Travis has an interesting grid view although I would concerned with database performance when rending this page.

Screen Shot 2020-03-13 at 6 33 13 PM

Also, we now have dedicated endpoints for fetching builds by branch 🎉. These endpoints are helpful because you are otherwise sourcing this data from the list of builds in the Vue store, which is going to be limited to the last 25 builds.

@jones2026
Copy link

@bradrydzewski would now be a good time to maybe cleanup some of the old branches so this is less confusing in the future?

@ashwilliams1
Copy link

@bradrydzewski I think we can probably migrate to master now right? @jones2026 I believe the reason for separate branches is that 0.8 used go get to pull the UI from master and for a period of time we were maintaining 0.8 even after 1.x was launched. Now that 0.8 is formally end of life I think it is safe to move the 1.x code to master. Do you agree Brad? If so I can make the change.

@bradrydzewski
Copy link
Member

Yes agreed, I went ahead and moved the vue branch to master. I will spend some time today trying to split the changes in vue-1.0.0 into separate feature branches so they can be worked on independently going forward.

@bradrydzewski
Copy link
Member

bradrydzewski commented Mar 19, 2020

We moved some of the code from vue-1.1.0 into master including the branch view (pictured below). You can see the interaction live here.

Screen Shot 2020-03-19 at 9 13 16 AM

Instead of having separate tabs to show all builds by branch, pull request, etc I was thinking we could provide an option to filter the main activity screen. Ideally the interaction would be really simple, for example:

Screen Shot 2020-03-19 at 10 41 12 AM

When clicking the filter button it could provide options to filter by common events and recent branches. The benefit of this sort of approach is simplicity. The event and branch fields in the database are indexed, so we would limit the filter to these fields.

Screen Shot 2020-03-19 at 9 38 53 AM

@NicsWorld
Copy link
Author

Hey @bradrydzewski thanks for all the info! II will shoot you a message on gitter shortly. I actually spiked out a filter menu similar to what you showed, i'd love to pursue or help anyway I can!

@bradrydzewski
Copy link
Member

Also see #317 which filters by branch. This could be expanded to filter by event type.

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.

4 participants