-
Notifications
You must be signed in to change notification settings - Fork 69
Automatically switch to post's organization on post show #384
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
Conversation
6c7d86e
to
5b47720
Compare
88e115f
to
8170499
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
cool hack @enricostano!
temporary they said...
@sseerrggii yep, much easier to implement this way. In any case we'll need to review this UI at some point. Can we leave it separated for now? EDIT: switch to |
Test a little more, everything seems to work fine 👍 |
WAT
Context: user that pertains to multiple time banks.
Right now visiting
/offers/666
the app will 💥 if the time bank the user selected is different than the requestedOffer
's organization.Until now this was fine, but since we're introducing mobile push notifications we'll need to deal with this case. The proposed solution is to witch to the
Post
's organization if the user is an active member of it.This is of course a temporary solution/hack ™️ until we finally tackle resources scoping under
/organization
.Test