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

Add custom pinned site #7644

Closed
NejcZdovc opened this issue Mar 10, 2017 · 9 comments
Closed

Add custom pinned site #7644

NejcZdovc opened this issue Mar 10, 2017 · 9 comments
Labels
addressed-with-brave-core Needs confirmation, but this issue may be resolved with Brave Core. feature/rewards needs-info Another team member needs information from the PR/issue opener. suggestion wontfix

Comments

@NejcZdovc
Copy link
Contributor

NejcZdovc commented Mar 10, 2017

Ability to add custom pinned site.
It's addition to the #7347 PR

When you click on Add, site is added, but dialog is not closed.
When you click on Done, dialog is closed and nothing is saved.

pasted image at 2017_03_09 03_42 pm

@mrose17
Copy link
Member

mrose17 commented Mar 10, 2017

i think this is a bad idea. when you pin a site because you've already gone there, we know (at least) that the URL is correct, etc.

second, what does it mean to pin a site that you never go to? that just seems... odd

@bradleyrichter - your thoughts?

@NejcZdovc
Copy link
Contributor Author

@mrose17 This issue was created on @bradleyrichter request, he also provided mock for it

@mrose17
Copy link
Member

mrose17 commented Mar 10, 2017

@NejcZdovc - thanks. i still think it's a bad idea, so i'm interested in hearing @bradleyrichter's thinking on this.

@bsclifton bsclifton added the needs-info Another team member needs information from the PR/issue opener. label Mar 17, 2017
@bsclifton
Copy link
Member

Assigning to @bradleyrichter so he can provide more info (or close issue if he agrees with @mrose17 )

@cndouglas
Copy link

What is the use case for this? We don't have this kind of functionality with bookmarks, for example.

@NejcZdovc
Copy link
Contributor Author

@bsclifton I think this is already defined. We defined it on the last ledger call. This is how you add custom publishers to the ledger. This publishers are pinned by default.

@cezaraugusto cezaraugusto added this to the Triage Backlog milestone Nov 8, 2017
@davidtemkin
Copy link

Where I could see this, or something similar, being very useful is if we were to provide a link to publishers/creators that, if viewed in Brave, would result in a dialog like this. Right now a publisher/creator who'd like to promote Brave (and seek contributions from Brave users) has to explain or link to an explanation of pinning (which despite our familiarity with it, is not exactly self-evident); and then instruct users to perform a set of manual steps.

@bsclifton bsclifton added wontfix addressed-with-brave-core Needs confirmation, but this issue may be resolved with Brave Core. labels Sep 3, 2018
@bsclifton bsclifton removed this from the Triage Backlog milestone Sep 3, 2018
@tildelowengrimm
Copy link

@bsclifton Do we have this functionality in the new version, or is it "addressed" in the sense that it's probably no longer needed because other functionality covers all the applicable use cases?

@bsclifton
Copy link
Member

@tomlowenthal I'm not sure of the specifics with Brave Rewards... but the entire system is overhauled. @NejcZdovc might know the answer offhand

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
addressed-with-brave-core Needs confirmation, but this issue may be resolved with Brave Core. feature/rewards needs-info Another team member needs information from the PR/issue opener. suggestion wontfix
Projects
None yet
Development

No branches or pull requests

8 participants