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

sorting - no order changed when it should #164

Closed
mmwo opened this issue Oct 9, 2016 · 3 comments
Closed

sorting - no order changed when it should #164

mmwo opened this issue Oct 9, 2016 · 3 comments
Assignees
Labels

Comments

@mmwo
Copy link

mmwo commented Oct 9, 2016

Hi, followed the tutorial to implement sort for subscribed collection.
Anyway the sort is not working as expected.
I tried the code from zip in step 13 and there is the same issue

Lets sey i have 3 parties

1. All have fun
2. Hey! Hi! Hello!
3. Public zone 

When I change the name of the party the view is updated with the new name but the order is as it was before.

I see in the view:

1. Zone <-- i expected this party to be at the end of the list
2. Hey! Hi! Hello!
3. Public zone

Party with changed name is not reordered on the list. Nothing helps to change it, jumping between routes, even page refresh. As if the name change was never done.

When I trigger sorting by myself i get the list like below:

1. Public zone 
2. Hey! Hi! Hello!
3. Zone <-- i expected this party to be on the top as it starts with 'Z' character

There is no error in console.

I would like to have my list to be refreshed when there is change of the name.
Can you help me solve it?

@dotansimha
Copy link
Collaborator

@jitra you are right.
The data need to trigger the sort change, verified it with a clean Meteor application.
I will update soon.

@dotansimha
Copy link
Collaborator

@jitra It's a bug in meteor-rxjs, and I fixes it in: Urigo/meteor-rxjs#21
We will soon merge, publish and update the tutorial :)

Keeping open until completely solved.

@dotansimha dotansimha self-assigned this Oct 25, 2016
@dotansimha
Copy link
Collaborator

@jitra
We fixed the bug in meteor-rxjs and released version 0.4.0 of it.
I also updated the tutorial with the latest version and now it works great!

Thanks again for spotting the bug!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants