Skip to content

Update index.js#10

Open
fabdbt wants to merge 1 commit intovictorsferreira:masterfrom
fabdbt:patch-1
Open

Update index.js#10
fabdbt wants to merge 1 commit intovictorsferreira:masterfrom
fabdbt:patch-1

Conversation

@fabdbt
Copy link

@fabdbt fabdbt commented Oct 25, 2017

Minor refacto using id instead of re-fetching all.
Allow to not redefining session-id.

Minor refacto using `id` instead of re-fetching `all`
@victorsferreira
Copy link
Owner

Hey man, thanks for your time.
I believe it helps only in terms of readability because id uses the method get that invokes getAll
Was that your intention?

@fabdbt
Copy link
Author

fabdbt commented Jan 25, 2018

Yes that's it, only in terms of readability. Just wanted to refactor code using corresponding method that fetch id instead of re-making same code. May be easier if session-id needs to be changed.

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.

2 participants