Skip to content
This repository was archived by the owner on Dec 5, 2024. It is now read-only.

Conversation

@StanleyGoldman
Copy link
Contributor

@StanleyGoldman StanleyGoldman commented Nov 9, 2017

  1. In preparation for Utilizing a cache for user data #427 "Adding cache to GitClient"... The GitClient will need to perform certain cache operations after the user is saved. In order to take a smaller step towards that goal I'm centralizing the functionality to set the git user and email in GitClient
  2. I also moved the definition of the name and email text boxes to a BeginChangeCheck/EndChangeCheck sequence. This ensures that needsSaving is only calculated when the textbox values have changed.

Depends on:

@StanleyGoldman StanleyGoldman force-pushed the fixes/git-client-set-user branch from 1f2c659 to 8b57bb9 Compare November 9, 2017 17:56
Copy link
Contributor

@jcansdale jcansdale left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wow, Unity UI programming takes some getting used to! This looks good from what I can work out. 👍

@StanleyGoldman StanleyGoldman merged commit 974b6b1 into master Nov 20, 2017
@StanleyGoldman StanleyGoldman deleted the fixes/git-client-set-user branch November 20, 2017 19:35
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants