Feature requests, bug reports, etc
Posted by
Ix Techau
almost 10 years
ago ·
299
replies
Use this thread to report any bugs you come across, or to suggest any new/fun features for the platform.
Last three updates (you can see a detailed changelog here):
v3.62 (19 Mar 2017)
- Removed the AAS due to low figures
- Some refactoring and code cleanup across the board
- Finally fixed the reset password system
- Removed all non-HTTPS links to avoid cross-region errors
- "Expected return" added to the Injury API
- Prediction performance and stats added to user profiles
v3.61 (28 Jan 2017)
- Code refactoring started, site should become faster over the next few weeks
- Complete revamp of the mobile site nav + mobile optimisations
- Style tweaks all over
v3.6 (26 Jan 2017)
- Small style tweaks and bug fixes all over
- Arsenal Report API v1 released!
Added +1 buttons. Right now they're not user limited, meaning a user can give a comment an infinite amount of +1s if he or she wants. This is intentional for now, but will probably change as the user base grows.
Posts
This badge was awarded for creating 200 posts
Comments
This badge was awarded for posting 1,000 comments
Polls
This badge was awarded for creating 10 polls
Membership
This badge was awarded for being member since December 2014
Bergkamp Award
This badge was awarded for being awesome
Admin
This badge was awarded for being an evil genius and coding this whole thing
Original Member
This badge was awarded for registering on Dial Square before 1 Jan 2015
Wait, what? You removed unread?
Comments
This badge was awarded for posting 100 comments
Polls
This badge was awarded for creating 5 polls
Membership
This badge was awarded for being member since January 2015
No it's still there, but only shows up if you actually have any unread posts.
Posts
This badge was awarded for creating 200 posts
Comments
This badge was awarded for posting 1,000 comments
Polls
This badge was awarded for creating 10 polls
Membership
This badge was awarded for being member since December 2014
Bergkamp Award
This badge was awarded for being awesome
Admin
This badge was awarded for being an evil genius and coding this whole thing
Original Member
This badge was awarded for registering on Dial Square before 1 Jan 2015
I made some suggestions on the previous page also, don't know if you saw them.
(Btw: the post you just made didn't show up in my unread topics. In fact the unread button didn't either)
Comments
This badge was awarded for posting 100 comments
Polls
This badge was awarded for creating 5 polls
Membership
This badge was awarded for being member since January 2015
Interesting. You still don't get an unread notice?
Posts
This badge was awarded for creating 200 posts
Comments
This badge was awarded for posting 1,000 comments
Polls
This badge was awarded for creating 10 polls
Membership
This badge was awarded for being member since December 2014
Bergkamp Award
This badge was awarded for being awesome
Admin
This badge was awarded for being an evil genius and coding this whole thing
Original Member
This badge was awarded for registering on Dial Square before 1 Jan 2015
The reason I made the polls the way they are is because I wanted them to feel almost standalone, and to be an introduction to Arsenal Report that looks slick and interesting. Most users viewing a poll would be non-members coming from say Reddit or Twitter, so I wanted a little wow-factor going so we can get more users in here!
Regarding comments on polls, that is coming soon. I just wanted to code them quickly and get them up there first, then we can expand its functionality. The back button is in there now if you've voted.
It would be pointless, as it would be an empty page. You can still bookmark it though, if that's what you want (/unread). I used to have the unread page as my starter page, but now my starter page is /news - it has all the info I need.
Newest page is gone (or well...you can still find it over at /newest, for now), the idea is that /discussions will be the one place for forum threads. Sorting options coming soon.
Posts
This badge was awarded for creating 200 posts
Comments
This badge was awarded for posting 1,000 comments
Polls
This badge was awarded for creating 10 polls
Membership
This badge was awarded for being member since December 2014
Bergkamp Award
This badge was awarded for being awesome
Admin
This badge was awarded for being an evil genius and coding this whole thing
Original Member
This badge was awarded for registering on Dial Square before 1 Jan 2015
Ooo actually I think I know what this is. Every day at 3am there's a background process running to clean up the unread markers, as they're taking up a lot of space in the database. I didn't think it was erasing all of them, which it probably does. I'll look into doing this once a week instead.
Posts
This badge was awarded for creating 200 posts
Comments
This badge was awarded for posting 1,000 comments
Polls
This badge was awarded for creating 10 polls
Membership
This badge was awarded for being member since December 2014
Bergkamp Award
This badge was awarded for being awesome
Admin
This badge was awarded for being an evil genius and coding this whole thing
Original Member
This badge was awarded for registering on Dial Square before 1 Jan 2015
I wouldn't say it would be a pointless button. I would click it every now and then to check for new content (since the page doesn't auto refresh, and I don't use F5 that frequently, any button that can refresh the page is better than the buildt-in browser-refresh button). And to be fair, I'd much rather have that button than an auto updating page (those are sometimes annoying).
In my opinion it is more pointless to remove the button as it could just as well be there (maybe grayed out or something) ;)
Comments
This badge was awarded for posting 100 comments
Polls
This badge was awarded for creating 5 polls
Membership
This badge was awarded for being member since January 2015
Hmm... This unread topic hassle seems extremely complex at the moment. Are you considering rebuilding it from the bottom if you come up with easier ways to do it? Or are you just trying to improve on the current system?
Comments
This badge was awarded for posting 100 comments
Polls
This badge was awarded for creating 5 polls
Membership
This badge was awarded for being member since January 2015
What I mean is, there is no purpose to have a visible link that leads to an empty state. You can always just go to /unread, it still works...but since it's completely empty, no need to link to it unless there is actually something there.
We had a custom built system before, which wasn't scalable. I then switched to a ready made solution recently, which works better. In addition, I've added a clean up process - this was probably the reason for you missing unread notices.
I think we still need a better solution. Right now a brand new member would get 260+ posts marked as unread, which isn't ideal. I think the best scenario in the future is that you only get unread notices on threads you've actually participated in, or subscribed to - similar to how Facebook works.
Posts
This badge was awarded for creating 200 posts
Comments
This badge was awarded for posting 1,000 comments
Polls
This badge was awarded for creating 10 polls
Membership
This badge was awarded for being member since December 2014
Bergkamp Award
This badge was awarded for being awesome
Admin
This badge was awarded for being an evil genius and coding this whole thing
Original Member
This badge was awarded for registering on Dial Square before 1 Jan 2015
The way some forums do it is by making all posts prior to your registration date marked as read, and any new activity after that is unread.
Posts
This badge was awarded for creating 5 posts
Comments
This badge was awarded for posting 100 comments
Polls
This badge was awarded for creating a poll
Membership
This badge was awarded for being member since January 2015
Yeah that's easy enough, just trigger a mark read upon registration...but I wonder how others handle the actual read marks. To me it's completely unscalable creating a view every time someone visits a thread. Right now it's fine, but let's say we have 200 members and 1,000 threads in a few months...that's 200,000 database entries just to handle what is unread or not.
Posts
This badge was awarded for creating 200 posts
Comments
This badge was awarded for posting 1,000 comments
Polls
This badge was awarded for creating 10 polls
Membership
This badge was awarded for being member since December 2014
Bergkamp Award
This badge was awarded for being awesome
Admin
This badge was awarded for being an evil genius and coding this whole thing
Original Member
This badge was awarded for registering on Dial Square before 1 Jan 2015
esreality.com has the best thread/unread comments system I have seen. It tells you the number of unread posts in a thread and unread posts are highlighted with a different background colour. The site was massive at one stage and has been around for 15 years or so with the same system so it cant be too taxing.
Comments
This badge was awarded for posting 50 comments
Membership
This badge was awarded for being member since April 2015
Yeah that's ideally how I'd want it to work as well, but I can imagine the difference being that their server costs were probably on a whole different tier than ours. The problem isn't that it can't be done, it's all just a matter of how much resource there is available.
I'm still looking into it, I'll have a good system working soon I just need to make the right decision that will scale if people start pouring in.
Posts
This badge was awarded for creating 200 posts
Comments
This badge was awarded for posting 1,000 comments
Polls
This badge was awarded for creating 10 polls
Membership
This badge was awarded for being member since December 2014
Bergkamp Award
This badge was awarded for being awesome
Admin
This badge was awarded for being an evil genius and coding this whole thing
Original Member
This badge was awarded for registering on Dial Square before 1 Jan 2015
This is how I've seen it done before...
Two models:
Topic(id: integer, name: string, created_at: datetime, updated_at: datetime etc...)
TopicView(id: integer, topic_id: integer, user_id: integer, created_at: datetime, updated_at: datetime)
When a user visits a topic a new TopicView record gets created specific to the topic_id and user_id combination.
If they've already visited the topic the TopicView record for that topic_id/user_id combination gets updated (updated_at field).
You can then check which topics are 'unread' for a user by comparing the updated_at datetimes on the TopicView and the Topic records. Additionally, filter out topics that have an updated_at datetime older than the user's created_at datetime.
LOL
Or pack it in and become a goat farmer?
ps. SEO friendly URLs for polls and players would be sassy.
Posts
This badge was awarded for creating 5 posts
Comments
This badge was awarded for posting 100 comments
Polls
This badge was awarded for creating a poll
Membership
This badge was awarded for being member since January 2015
This is what I had before, but isn't it extremely database-heavy? I'm far from a Rails expert, but as far as I can tell, using the system of a unique view per thread per user means we'll have huge amounts of database rows?
500 users * 1,000 discussion threads = 500,000 topic views = £1bn per second hosting costs. ;)
And if I clean them up once a week say, everything will now become unread for everyone.
Using this at the moment: https://github.com/ledermann/unread, which I think works similarly.
Yup, definitely.
Posts
This badge was awarded for creating 200 posts
Comments
This badge was awarded for posting 1,000 comments
Polls
This badge was awarded for creating 10 polls
Membership
This badge was awarded for being member since December 2014
Bergkamp Award
This badge was awarded for being awesome
Admin
This badge was awarded for being an evil genius and coding this whole thing
Original Member
This badge was awarded for registering on Dial Square before 1 Jan 2015
Well it wouldn't be 1,000 topic views per user... I assume you'd only create the record if the topic had been viewed.
Might be more like 500 users * 100 topic views per average user = 50,000 records. But I get where you're going... it could create a huge number of rows in the long term.
I have a database table with 75 million rows in and it performs ok on these kind of joins.
That gem looks good. There's a little mention of performance "Hint: You should add a database index on messages.created_at"
Posts
This badge was awarded for creating 5 posts
Comments
This badge was awarded for posting 100 comments
Polls
This badge was awarded for creating a poll
Membership
This badge was awarded for being member since January 2015
Yeah I have yet to figure out how to add indexes on already made attributes. I know how to do it when creating them (user_id:integer:index, etc).
Posts
This badge was awarded for creating 200 posts
Comments
This badge was awarded for posting 1,000 comments
Polls
This badge was awarded for creating 10 polls
Membership
This badge was awarded for being member since December 2014
Bergkamp Award
This badge was awarded for being awesome
Admin
This badge was awarded for being an evil genius and coding this whole thing
Original Member
This badge was awarded for registering on Dial Square before 1 Jan 2015
Yes, but...if someone marks all posts as read, a view would be created for every post in history, I assume?
Posts
This badge was awarded for creating 200 posts
Comments
This badge was awarded for posting 1,000 comments
Polls
This badge was awarded for creating 10 polls
Membership
This badge was awarded for being member since December 2014
Bergkamp Award
This badge was awarded for being awesome
Admin
This badge was awarded for being an evil genius and coding this whole thing
Original Member
This badge was awarded for registering on Dial Square before 1 Jan 2015
Fair point
Posts
This badge was awarded for creating 5 posts
Comments
This badge was awarded for posting 100 comments
Polls
This badge was awarded for creating a poll
Membership
This badge was awarded for being member since January 2015
http://stackoverflow.com/questions/15881749/adding-index-after-adding-column-rails
Posts
This badge was awarded for creating 5 posts
Comments
This badge was awarded for posting 100 comments
Polls
This badge was awarded for creating a poll
Membership
This badge was awarded for being member since January 2015
You are not logged in!
Join Arsenal Report today to improve your experience using the site with thread subscriptions + custom profile with cover image and favourite XI + the ability to post comments, polls and AMA questions.