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

update "unread" message in dashboard #116

Closed
pruffier opened this issue Nov 29, 2017 · 7 comments
Closed

update "unread" message in dashboard #116

pruffier opened this issue Nov 29, 2017 · 7 comments
Labels

Comments

@pruffier
Copy link

pruffier commented Nov 29, 2017

Q A
Bug report? yes
Feature request? no
Support request? yes

hello,
having issues with the message within the admin panel. when reading the message, it does not update my dashboard notification of the unread message. it keeps my "dashboard (1)..."
thank you!

@cocolabssas
Copy link
Contributor

Hi,

It means you still have a unread message in the offerer or asker dashboard. Reading your messages in both dashboards will removed the remaining unread message.

@pruffier
Copy link
Author

pruffier commented Dec 8, 2017

hi,

no, have read from the offerer and asker dashboard and it does not update.

thanks.

@eshelf
Copy link

eshelf commented Dec 15, 2017

Same problem for me. The number of unread messages is not updated after reading the messages.

@cocolabssas
Copy link
Contributor

Hi,

It is a problem of Doctrine result cache. We are fixing it.

@eshelf
Copy link

eshelf commented Dec 24, 2017

Hi, thanks for updating us! The issue was closed. How will us know when the bug is fixed?

@eshelf
Copy link

eshelf commented Dec 29, 2017

Hey, guys! Thanks for dealing with this.

I've tried it and I think you guys also need to include the line below in the methods indexAction, deleteAction and newThreadAction.

$this->get('doctrine')->getManager()->getRepository('CocoricoMessageBundle:Message') ->clearNbUnreadMessageCache($this->getUser()->getId());

Besides that, in the method nbUnReadMessagesAction, only GET was accepted and it was generating an error. I had to delete it in order to properly function.

I'm sorry for not sending a pull request. As you can see on issue 126, I'm strugling to sincronize my repository with yours.

@cocolabssas
Copy link
Contributor

Hi eshelf,

Thanks for your feedback.
The doctrine result cache is now cleared for new and deleted messages (f55522e).

Concerning the nbUnReadMessagesAction only the method GET should be used and we have no errors with this method.

@cocolabssas cocolabssas reopened this Jan 3, 2018
cocolabssas added a commit that referenced this issue Mar 20, 2018
### Added
- Add minimal tests
- Add multi parameters on Booking validator messages
- Add static property access twig function
- Add Booking in Admin Review
- Add security voter to voucher page access
- Add Carrier bundle hatchback field to booking admin
- Add listing search engine on user profile page
- Add error message on listing calendar edition
- Set booking status to STATUS_PAYMENT_REFUSED when booking can not be validated
- Add booking acceptation delay
- Add listing show query overriding
- Add parameter type in CocoricoConfigBundle
- Add LISTING_SEARCH_HIGH_RANK_QUERY and LISTING_SEARCH_BY_IDS_QUERY events

### Fixed
- Fix js escaping on result
- Fix #116 issue (update "unread" message in dashboard)
- Fix Booking expiration alert
- Replace depreciated 'intention' by 'csrf_token_id' in BookingNewType
- Fix search categories field displaying
- Fix #135 issue error when add to favourite
- Fix booking validation date by adding time to date verification
- Fix discount unicity issue while adding / removing discounts
- Fix similar listings
- Fix favorites listings
- Fix googlemaps MarkerWithLabel issue 393

### Changed
- Update doc
- Move some GlobalHelper methods to Utils\PHP class
- Handle translator provider errors
- Change `cocorico.booking.min_start_time_delay` parameter unit from hours to minutes
- Add minor corrections to ReviewBundle
- Disable listing deposit and new booking to admin user
- Add phone to registration form
- Add phone and email verification message on profile edition
- Decouple SMSBundle
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

3 participants