Skip to content
This repository was archived by the owner on Jun 27, 2018. It is now read-only.

[RFR] Add parameter in View.getReferences to filter on refreshDelay#19

Merged
jeromemacias merged 5 commits intomasterfrom
reference_refresh_delay
Jun 26, 2015
Merged

[RFR] Add parameter in View.getReferences to filter on refreshDelay#19
jeromemacias merged 5 commits intomasterfrom
reference_refresh_delay

Conversation

@jpetitcolas
Copy link
Copy Markdown
Contributor

Useful to prevent from doing an extra request at route resolving on ng-admin (see marmelab/ng-admin#508).

@jpetitcolas jpetitcolas changed the title [RFR] Add parameter in View.getReferences to filter on refreshDelay [WIP] Add parameter in View.getReferences to filter on refreshDelay Jun 25, 2015
@jpetitcolas
Copy link
Copy Markdown
Contributor Author

Back to WIP. As seen with @fzaninotto, we will use an autcomplete method taking an option for refreshDelay.

@jpetitcolas
Copy link
Copy Markdown
Contributor Author

Switching to RFR.

@fzaninotto, I added two methods autocomplete and autocompleteOptions to be more consistent with other methods: they always take a single parameter when used as setter.

@jpetitcolas jpetitcolas changed the title [WIP] Add parameter in View.getReferences to filter on refreshDelay [RFR] Add parameter in View.getReferences to filter on refreshDelay Jun 25, 2015
@jpetitcolas
Copy link
Copy Markdown
Contributor Author

After more discussions, still wrong naming. We are going to use remoteComplete([true|false], options) instead. Back to WIP.

@jpetitcolas jpetitcolas changed the title [RFR] Add parameter in View.getReferences to filter on refreshDelay [WIP] Add parameter in View.getReferences to filter on refreshDelay Jun 25, 2015
@jpetitcolas
Copy link
Copy Markdown
Contributor Author

Back to RFR.

@jpetitcolas jpetitcolas changed the title [WIP] Add parameter in View.getReferences to filter on refreshDelay [RFR] Add parameter in View.getReferences to filter on refreshDelay Jun 26, 2015
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Missing @todo

lib/View/View.js Outdated
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I think you should rename withRefreshDelay to withRemoteComplete.

@jpetitcolas
Copy link
Copy Markdown
Contributor Author

Reviewed

jeromemacias added a commit that referenced this pull request Jun 26, 2015
[RFR] Add parameter in View.getReferences to filter on refreshDelay
@jeromemacias jeromemacias merged commit 78024b4 into master Jun 26, 2015
@jeromemacias jeromemacias deleted the reference_refresh_delay branch June 26, 2015 09:13
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