Skip to content
This repository has been archived by the owner on Dec 11, 2019. It is now read-only.

History not being updated on first-time site visit #5542

Closed
cezaraugusto opened this issue Nov 10, 2016 · 7 comments
Closed

History not being updated on first-time site visit #5542

cezaraugusto opened this issue Nov 10, 2016 · 7 comments

Comments

@cezaraugusto
Copy link
Contributor

cezaraugusto commented Nov 10, 2016

Did you search for similar issues before submitting this one?
Yes

Describe the issue you encountered:
When visiting a website, about:history should be updated with each new access, which is not happening unless you refresh the page (2nd access).

Expected behavior:
About:history should be updated on the first website visit.

  • Platform (Win7, 8, 10? macOS? Linux distro?): n/a

  • Brave Version: 0.10.9

  • Steps to reproduce:

    1. Open Brave
    2. Visit a website
    3. Check about:history
    4. Visited site is not there
    5. Go back to website, refresh page
    6. Site is updated on about:history
  • Screenshot if needed: n/a

  • Any related issues:
    Not really related to the issue but being affected: Website not getting added to the top site tile upon first visit #5511

@Sh1d0w
Copy link

Sh1d0w commented Nov 10, 2016

Possibly related or duplicate issue #4794

@bsclifton
Copy link
Member

Haven't had time to look at this yet ☹️ I'll move to 0.12.11

I'll un-assign myself for now (please feel free to grab this!). Will check out next week

@bsclifton bsclifton removed their assignment Nov 17, 2016
@bsclifton
Copy link
Member

@cezaraugusto this might be a good one for you to check out- let me know if you're interested

@bsclifton bsclifton modified the milestones: 0.12.11, 0.12.10 release Nov 17, 2016
@cezaraugusto
Copy link
Contributor Author

Taking

@cezaraugusto cezaraugusto self-assigned this Nov 17, 2016
@bsclifton bsclifton assigned bsclifton and unassigned cezaraugusto and bsclifton Nov 17, 2016
@bbondy bbondy modified the milestones: Backlog, 0.12.11 Nov 18, 2016
@bbondy
Copy link
Member

bbondy commented Nov 18, 2016

I think it eventually does show so moving to backlog for now, would love a fix.

@NejcZdovc
Copy link
Contributor

@cezaraugusto Can you please check if this is already fixed?

@cezaraugusto
Copy link
Contributor Author

@NejcZdovc good find, not an issue anymore.

@luixxiul luixxiul removed this from the Backlog milestone Jul 22, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants