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

Field Guide doesn't work #367

Closed
kmiasko opened this issue Apr 25, 2015 · 1 comment
Closed

Field Guide doesn't work #367

kmiasko opened this issue Apr 25, 2015 · 1 comment
Assignees
Labels
type: bug Issues that need priority attention. Platform, Curriculum tests (if broken completely), etc.

Comments

@kmiasko
Copy link
Contributor

kmiasko commented Apr 25, 2015

Completed Field Guides array has been cleared (filled with only first Field Guide id). User can show all Field Guides but can't complete any of them.

if (data.completedFieldGuides.indexOf(data.fieldGuideIds[i]) > -1) {
will always return -1 because data.fieldGuideIds[i]._id should be used.

After clicking 'Next Field Guide' button no Post request is fired to mark field guide as completed.

@terakilobyte terakilobyte added the type: bug Issues that need priority attention. Platform, Curriculum tests (if broken completely), etc. label Apr 25, 2015
@terakilobyte terakilobyte self-assigned this Apr 25, 2015
@terakilobyte
Copy link
Contributor

Thanks for the report. This was squashed previously but we're experiencing zombie code.

@terakilobyte terakilobyte added status: waiting review To be applied to PR's that are ready for QA, especially when additional review is pending. and removed in progress labels Apr 25, 2015
@QuincyLarson QuincyLarson removed the status: waiting review To be applied to PR's that are ready for QA, especially when additional review is pending. label Apr 26, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: bug Issues that need priority attention. Platform, Curriculum tests (if broken completely), etc.
Projects
None yet
Development

No branches or pull requests

3 participants