macgitver / MacGitverModules

DEPRECATED: Modules for MacGitver
5 stars 1 forks source link

fetch remotes of a repository #101

Closed antis81 closed 9 years ago

antis81 commented 9 years ago

Some thoughts about this PR: It is not a good idea to realize the remotes menu in a central place. Why? Because from a users point of view there is no obvious context (What am I fetching/pushing from/to where? To which repository will a new remote belong?). Also the options to the fetch process are context-specific. The actions should actually be added to the appropriate context menus. For example a "fetch action" on a remote branch, fetches the remote branch. IOW: This belongs to the Reference-Tree and probably to the inline remote references in the History-View (we don't have a context menu for them yet). The actions itself should be defined central in the "Remotes" module, which should provide a the generic remote operations and just needs to be attached to the appropriate context. For this reason, the concept of the DAM (Dynamic Action Merger) has to be fully implemented first. This PR has to be hold back until then.

This is how it could look like? mgv-fetch-repo-ctx-menu

Planned for this PR:

Excluded from this PR

More Ideas:

antis81 commented 9 years ago

@scunz This is ready to merge.

EDIT: FYI. To make life easier, I moved this PR from sc/refsview back to development. It will not compile, due to the small changes, that went to the wrong place. I don't care and move on anyways.

scunz commented 9 years ago

No, this is not ready to merge. Please hold off until I finished my mail - then you'll understand why - but it's taking ages to do that :)

antis81 commented 9 years ago

Yeah, I know this is unfinished stuff. I just want to trigger the merge as soon as possible, to help getting the submodule integration done quickly, before this gets into a more complex discussion and delaying things until next year - ya know what I mean :).

antis81 commented 9 years ago

Closing, as this is a POC, not a final implementation. Moved to new PR #104 taking a different approach to abstract remote operations in a separate MacGitver module.

EDIT: I'll keep the branch to not lose the code.