incubator-allura-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Igor Bondarenko" <jetm...@users.sf.net>
Subject [allura:tickets] #6446 Forum API should check for posts' status
Date Thu, 25 Jul 2013 09:27:36 GMT
Closed #400. `je/42cc_6446`


---

** [tickets:#6446] Forum API should check for posts' status**

**Status:** code-review
**Labels:** 42cc allura-api 
**Created:** Wed Jul 10, 2013 03:19 PM UTC by Dave Brondsema
**Last Updated:** Thu Jul 11, 2013 08:36 AM UTC
**Owner:** nobody

The new API in the ForgeDiscussion tool doesn't check for posts' statuses, so it will show
'pending' and 'spam' posts too, which aren't shown on the regular forum view.  This may apply
to discussion thread APIs in other tools too (e.g. ticket comments via API).  It's an issue
for individual threads URLs and also the API endpoint for a forum which lists the threads
(a new thread that is just 1 post that is not "ok" status, shouldn't be listed, just like
the web interface)

Perhaps easiest would be to only show "ok" posts no matter what.  Would be a bit nicer if
a user with the proper rights could see the pending & spam posts too.  That might be best
with an additional flag in the URL to control whether to show those?  Doesn't seem necessary
to do that extra stuff at this point though.


---

Sent from sourceforge.net because allura-dev@incubator.apache.org is subscribed to https://sourceforge.net/p/allura/tickets/

To unsubscribe from further messages, a project admin can change settings at https://sourceforge.net/p/allura/admin/tickets/options.
 Or, if this is a mailing list, you can unsubscribe from the mailing list.
Mime
  • Unnamed multipart/related (inline, None, 0 bytes)
View raw message