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

Separate new accounts/lists from rest of 'recent changes' 'by humans #2640

Open
BrittanyBunk opened this issue Nov 14, 2019 · 11 comments
Open
Labels
Affects: UI Issues with the web site's user interface. [managed] Lead: @jdlrobson Issues overseen by Jon (Front-end Lead) [managed] Priority: 3 Issues that we can consider at our leisure. [managed] Type: Feature Request Issue describes a feature or enhancement we'd like to implement. [managed]
Projects

Comments

@BrittanyBunk
Copy link
Contributor

https://openlibrary.org/recentchanges?#humans

@BrittanyBunk BrittanyBunk added the Type: Feature Request Issue describes a feature or enhancement we'd like to implement. [managed] label Nov 14, 2019
@BrittanyBunk
Copy link
Contributor Author

BrittanyBunk commented Nov 14, 2019

it may help with preventing people from writing over each other's work and also increases the capability for double-checking/adding on more info

@BrittanyBunk
Copy link
Contributor Author

BrittanyBunk commented Nov 14, 2019

yes, that would be good. Where does one access those? Could that be added to the main page I linked?

@xayhewalo xayhewalo added this to Un-Triaged in Triage Nov 15, 2019
@xayhewalo
Copy link
Collaborator

xayhewalo commented Nov 26, 2019

So is this about making an existing feature more accessible or adding new functionality? It looks like recent lists made by humans exists or am I misunderstanding the question?

@xayhewalo xayhewalo added Needs: Triage This issue needs triage. The team needs to decide who should own it, what to do, by when. [managed] Priority: 3 Issues that we can consider at our leisure. [managed] State: Backlogged labels Nov 26, 2019
@xayhewalo xayhewalo moved this from Un-Triaged to Needs: Assessment in Triage Nov 26, 2019
@tfmorris
Copy link
Contributor

Tagging it as a feature enhancement is fine. Invisible features don't really count. It's just a matter of adding a few more labels at the top of the page with links to the URLs I posted.

@BrittanyBunk
Copy link
Contributor Author

BrittanyBunk commented Nov 27, 2019

@guyjeangilles on https://openlibrary.org/recentchanges, the 1st link @tfmorris shows is not on that page. It's valuable to me to add there.

@BrittanyBunk
Copy link
Contributor Author

@guyjeangilles @cdrini I was wondering if we could revisit this, as it's a really simple fix to add the one link ('edit books') to the recent changes page. Then it could get closed. That is, if everyone thinks is good to add. I think it's really helpful to see the actual changes happening that are not just everything (which is mostly accounts and lists created). It would be called "book edits" and would be the first on the list before 'author merges'

@BrittanyBunk
Copy link
Contributor Author

I don't think that'll make the issue closed, as then there are still the 'new accounts' to add too. Those two are what's left.

@xayhewalo xayhewalo added Affects: UI Issues with the web site's user interface. [managed] Lead: @jdlrobson Issues overseen by Jon (Front-end Lead) [managed] labels Dec 27, 2019
@xayhewalo xayhewalo removed the Needs: Triage This issue needs triage. The team needs to decide who should own it, what to do, by when. [managed] label Jan 25, 2020
@SouthGoingZax
Copy link
Collaborator

@BrittanyBunk I sent #4137 to add a link to recent book edits, but I didn't add a link to new accounts because of #4136. My experience with /recentchanges is that it's usually flooded with new account creations, so I don't think there's a lack of visibility there. Given that, I'd be fine closing this issue once/if the referenced PR is merged. What do you think?

@BrittanyBunk
Copy link
Contributor Author

BrittanyBunk commented Nov 23, 2020

@SouthGoingZax hey, the fact that you're moving this forward means a lot. I kind of agree that we wouldn't really need a /newaccounts tab, but hey, separation never hurts (unless it's too much effort)! Hope that helps. So far, really great work and this is what I'm hoping for! It's a really handy tab that I use a lot and bet others will too (the edit book tab/page especially is what I mean).

@SouthGoingZax
Copy link
Collaborator

It's a one-line change to link to /recentchanges/new-account. Given that it often times out (#4136), I thought it might not make sense to link to it yet.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Affects: UI Issues with the web site's user interface. [managed] Lead: @jdlrobson Issues overseen by Jon (Front-end Lead) [managed] Priority: 3 Issues that we can consider at our leisure. [managed] Type: Feature Request Issue describes a feature or enhancement we'd like to implement. [managed]
Projects
No open projects
Triage
  
Needs: Assessment
Development

No branches or pull requests

4 participants