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

Initial Release for My Tickets #171

Closed
wants to merge 0 commits into from
Closed

Conversation

Hybris95
Copy link
Contributor

Simple "My Tickets" view.
Can be accessed via /mytickets (yet).

@gitblit
Copy link
Collaborator

gitblit commented Apr 16, 2014

I'll take a closer look at this after the 1.5.0 release which should be out by the end of the week.

@Hybris95
Copy link
Contributor Author

Should I rebase my commits when 1.5.0 is out or will you manage to rewrite the commits ?

@gitblit
Copy link
Collaborator

gitblit commented Apr 16, 2014

Sure, why not.

@Hybris95
Copy link
Contributor Author

Commits were rebased.

@gitblit
Copy link
Collaborator

gitblit commented Apr 22, 2014

Hi Christian,

I think the My Tickets page is a pretty large and complex feature - that is why I didn't include it in 1.4.0. It would have delayed me several more weeks. I've pushed a branch with your commits plus a revision commit where I have revised some of your approach. It is still very rough/primitive and a long way from mergeable. We should be aiming to have something as functional as the standard Tickets page, but streamlined for things relevant to the current user.

Things to consider in My Tickets....

  1. anonymous users are irrelevant - there is no "my" for an anonymous user. We should redirect AND not bother showing the link for anon users.
  2. repositories with work trees can be ignored
  3. we should use the Lucene index rather than mining tickets directly - that won't scale.

The Lucene index does not keep track of who votes, so we can't query for your voted tickets, but all other ticket attributes should be represented.

It might be helpful for you to have real ticket data when working on this. One way you can do this is to clone my repo - but you'll have to use the BranchTicketService.

git clone --mirror https://dev.gitblit.com/r/gitblit.git

After you clone, run the ReindexTickets tool. This will reindex all tickets from all repos using your configured service - which should be BranchTicketService, if using my data.

-J

@Hybris95
Copy link
Contributor Author

I forked the branch "mytickets" on my own repo, and squashed all the commits done until then since you imported them in your own "mytickets" branch.

I think this pull request can be closed.

I will work soon on improving my initial system with the directives you gave me.

@Hybris95 Hybris95 closed this Apr 22, 2014
gitblit added a commit that referenced this pull request Sep 8, 2014
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.

None yet

2 participants