Skip to content
This repository was archived by the owner on Feb 12, 2022. It is now read-only.

Refactoring the naming of configuration objects#4

Open
hemmeter wants to merge 1 commit intoforcedotcom:masterfrom
hemmeter:master
Open

Refactoring the naming of configuration objects#4
hemmeter wants to merge 1 commit intoforcedotcom:masterfrom
hemmeter:master

Conversation

@hemmeter
Copy link

Refactoring the naming of configuration objects to make it easier to load into an org with existing config so name conflicts are minimized.

  • Renamed the objects to prefix them with "UserAccess".
  • Updated all the code references to use the new names
  • 1 small change to the page to render based upon the UID value being present in the URL.

…load into an org with existing config so name conflicts are minimized.

- Renamed the objects to prefix them with "UserAccess".
- Updated all the code references to use the new names
- 1 small change to the page to render based upon the UID value being present in the URL.
@sfdcdoug-zz
Copy link

I see you snuck a usability nicety in there with an actual error message when the uid param isn't provided. Nice. :)

@hemmeter
Copy link
Author

hemmeter commented Jan 4, 2014

Is this going to be committed to master?

@sfdcdoug-zz
Copy link

It's amazing how quickly things get lost on my TODO list. :) Ping me if you don't see this merged in this week. Slightly more complex of a pull than I'd like to do while on BART.

@hemmeter
Copy link
Author

hemmeter commented Jan 6, 2014

No rush at all. Just checking in on it.

On Mon, Jan 6, 2014 at 7:49 AM, sfdcdoug notifications@github.com wrote:

It's amazing how quickly things get lost on my TODO list. :) Ping me if
you don't see this merged in this week. Slightly more complex of a pull
than I'd like to do while on BART.


Reply to this email directly or view it on GitHubhttps://github.com//pull/4#issuecomment-31658381
.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants