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

Fetch current user's activity in CampaignView.componentDidMount #844

Merged
merged 5 commits into from
Feb 23, 2016

Conversation

aaronschachter
Copy link
Contributor

Queries the signups endpoint upon loading a CampaignView, and adds case to listen for a Reportback object sent from the LDTAppDelegate.bridge and display the Self Reportback accordingly (closes #757). This prevents a scenario where the User may signup for a campaign on the web after the app launches and an older campaign status is cached (user would have need to force quit).

Removes the DSOUser.campaignSignups property - was running into a lot of trouble keeping this up to date, and not worth the headache of parsing through inactive campaign runs locally for not much gain. This allows us to simplify things and kill lots of code! 🔥

Refs #807 - completes Doing section for public profile, just need to add Share Photo button (see below)

Closes #814

TODO:

  • Share your photo button needs to work, and get added to the Self Profile as well

aaronschachter added a commit that referenced this pull request Feb 23, 2016
Fetch current user's activity in CampaignView.componentDidMount
@aaronschachter aaronschachter merged commit b3889a3 into develop Feb 23, 2016
@aaronschachter aaronschachter deleted the deepEnd branch February 23, 2016 01:00
@aaronschachter aaronschachter mentioned this pull request Feb 23, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

API changes for user/:id/campaigns Use campaign current_run id to determine user's current activity
1 participant