Skip to content
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

allow for polling and updating of cards while viewing a board #5

Merged
merged 1 commit into from
Oct 25, 2016

Conversation

boom
Copy link
Contributor

@boom boom commented Oct 25, 2016

No description provided.


function checkForQuery() {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sorry for the refactor. Since the new functionality depends on jQuery as well, figure I'd refactor the check to ensure its ready so it can be used to trigger off multiple methods.

}
}

function improveCollaborationBox() {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Only change to this function is the method name and the indentation. Sorry for the diff noise


function updateStages(newStages) {
stages = newStages;
Backbone.history.loadUrl();
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This is the magic. Since blossom uses backbone, we can reload the current route. This triggers just a refetching of the cards.

Copy link
Owner

@tonybaroneee tonybaroneee left a comment

Choose a reason for hiding this comment

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

Awesome work boom!

@tonybaroneee tonybaroneee merged commit d26bdaf into tonybaroneee:master Oct 25, 2016
@boom boom deleted the feature/add-polling-support branch October 25, 2016 20:55
@boom boom restored the feature/add-polling-support branch October 25, 2016 20:57
@boom boom deleted the feature/add-polling-support branch October 25, 2016 20:58
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