-
Notifications
You must be signed in to change notification settings - Fork 123
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
[WIP] Mac refactor demo 2019 #771
Conversation
- add synctriplet - move sync repo independent package - move config to independent package - sync machine todo
…iting list to handle this case
…der's remote path
…ate new file or folder under the deleted one
… add new file problem
get parent of file in the root folder's bug
|
||
namespace CmisSync.Lib.Sync.SyncMachine | ||
{ | ||
public class SyncTripletProcessor |
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.
Class documentation would be great :-)
return res; | ||
} | ||
|
||
public static SyncTriplet CreateSFGFromLocalFolder( |
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.
Expanding SFG (or at least explaining it) would make the code more understandable to readers.
<Compile Include="Config\CmisSyncConfig.cs" /> | ||
<Compile Include="Config\ConfigManager.cs" /> | ||
<Compile Include="Config\ConfigMigration.cs" /> | ||
<Compile Include="Sync\SynchronizeItem\SyncItem.cs" /> |
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.
SynchronizeItem
-> SynchronizedItem
?
metadata = OperationUtils.FetchMetadata (remoteDocument, session); | ||
} catch (CmisBaseException e) { | ||
// Remove temporary local document to avoid it being considered a new document. | ||
Console.WriteLine (" %% download error, " + e.Message); |
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.
Why not to the log file?
Based on 2017/12's old version.
Crawlsync tested, changlog sync tested for basic behavior. Local watcher not implemented yet