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

Add-on Store: "Community reviews" action should be available in Installed add-ons tab too #16179

Closed
CyrilleB79 opened this issue Feb 14, 2024 · 32 comments · Fixed by #16368
Closed
Labels
good first issue github features these at https://github.com/nvaccess/nvda/contribute p4 https://github.com/nvaccess/nvda/blob/master/projectDocs/issues/triage.md#priority quick fix triaged Has been triaged, issue is waiting for implementation.
Milestone

Comments

@CyrilleB79
Copy link
Collaborator

Is your feature request related to a problem? Please describe.

"Community review" action is now available in beta for some weeks. Though people do not seem to use it.

There are two use cases to use it:

  • read reviews from other users to decide to install (or update) an add-on or not
  • write a feedback to let other people know what you think of an add-on that is already installed.

Or maybe sometimes people open the reviews page to gather opinions on an add-on but these pages are desperately empty.

I wonder if one reason why the feature is not used is that it is not visible enough.

Describe the solution you'd like

To encourage the second use case, i.e. writing feedback, I'd ask that the action also appears in the context menu of add-ons in the "Installed add-ons" tab.

Indeed, if I am satisfied with the add-on, I will interact with it in the "Installed add-ons" tab.
If I am not satisfied, I will also launch the removal from the Installed add-on tab.
At last, the Installed add-ons tab is the most important because it's the one that open when starting the add-on store.

Describe alternatives you've considered

Other solutions to promote the feature may be:

  • a better integration of the feature in the store, i.e. have the review page displayed in the store, without the whole GitHub menu
  • An automatic translation of the reviews page with a translation widget, automaticially using NVDA's language.
  • advertising the feature on the lists or in In-process

Additional context

The feature will be launched generally with NVDA 2024.1. I'd suggest that this request be integrated as soon as the feature is available, i.e. have it implemented in beta (cc @seanbudd). Indeed, it would mitigate the risk of a bad first impression.

Also cc @nvdaes

@nvdaes
Copy link
Sponsor Contributor

nvdaes commented Feb 14, 2024

I think that the main reason because this is not used is that reviewing add-ons implies a great responsibility and it's something hard to do. Furthermore,there's no standardized process and people may not be sure about what to include in reviews.
For me, the best way to promote this is to participate in add-ons review. I reviewed Windows App Essential and MathCAT.
But imo this shouldn'tbe included in the installed tab,since add-ons from externalsources cannot be reviewed in the store. GitHub has buttons to translate discussion.This maybe promotedin In-Process, @Qchristensen . Anyway, for example I use just a few add-ons, and many of them are maintained by me, so I shouldn't review other add-ons. I think that this won't be very used since reviewing add-ons is really difficult. But regarding your request I think that the only tabs where it can be implemented for all add-ons are Available and updatable.

@CyrilleB79
Copy link
Collaborator Author

I am not sure to understand what is expected in this reviews discussion:

  • opinions on the add-on, such as the comments that we can find on Google Play Store
  • or an add-on review, such as what was required a few years ago to accept an add-on on the community website

@nvdaes
Copy link
Sponsor Contributor

nvdaes commented Feb 15, 2024 via email

@CyrilleB79
Copy link
Collaborator Author

OK, thanks. So I had understood the concept correctly.

My proposal is that the action also appears in the Installed add-on tab, but also still remain in the available tab (and the updatable tab). To summarize, I want to add it in one more tab.

@nvdaes
Copy link
Sponsor Contributor

nvdaes commented Feb 16, 2024 via email

@XLTechie
Copy link
Collaborator

XLTechie commented Feb 16, 2024 via email

@nvdaes
Copy link
Sponsor Contributor

nvdaes commented Feb 16, 2024 via email

@XLTechie
Copy link
Collaborator

XLTechie commented Feb 16, 2024 via email

@nvdaes
Copy link
Sponsor Contributor

nvdaes commented Feb 16, 2024

If an add-on has been removed from the store, what prevents a user from
reading reviews of it?

Nothing. You have convinced me with this argument.
Anyway, we should promote this feature, since working on it if the existing feature is not very used may not be grateful. And for me the main reason cause people don't use this is that it's a very difficult task.

@CyrilleB79
Copy link
Collaborator Author

@nvdaes you write:

I think that the main reason because this is not used is that reviewing add-ons implies a great responsibility and it's something hard to do. Furthermore,there's no standardized process and people may not be sure about what to include in reviews. For me, the best way to promote this is to participate in add-ons review.

and:

according to the user guide, users can provide feedback, so this is not anything technical. In the Windows App Essential addon I left an horay reaction,, and in mathCath, I higligted the support for braille CMU.

and also:

And for me the main reason cause people don't use this is that it's a very difficult task.

Could we clarify what is difficult in the task of writing reviews? @nvdaes have you something very specific in mind?

Having an opinion on an add-on does not seem something very difficult: there are many people talking of add-ons on the mailing lists, and why they love them or not.

@nvdaes
Copy link
Sponsor Contributor

nvdaes commented Feb 16, 2024

Could we clarify what is difficult in the task of writing reviews? @nvdaes have you something very specific in mind?

Maybe my own experience from about 2013. Maybe comment from people saying that they didn't want to receive comments about how to write add-ons, or people making bad criticism about reviewers who say that a particular add-on maybe dangerous. Maybe my own experiences. Hope I'm wrong. But the fact is that this is not used, I think.
For me reviewing add-ons is difficult since I try to look at the code, total or partially, though I don't provide technical opinions. I have dedicated many hours to review add-ons in the past, so I may be talking about myself 😃

@CyrilleB79
Copy link
Collaborator Author

But we are mixing two things here.
I was not active in 2013 in NVDA add-ons community; but I guess it was more a matter of technical review. The community reviews action instead is meant to be opinions from user, as I understand it.

@nvdaes
Copy link
Sponsor Contributor

nvdaes commented Feb 16, 2024

The community reviews action instead is meant to be opinions from user, as I understand it.

Yes, but if a person can analize the source code, it may be considered in the opinion.
At least personally I look at the code, in case performing the proposed commands, for example, has something not expected and not shown inmediately.

@nvdaes
Copy link
Sponsor Contributor

nvdaes commented Feb 16, 2024

Also, if a person doesn't like an add-on, probably she or he won't use it and so it won't be reviewed.
I'm thinking about an add-on that for me would get a bad review, but simply I won't use it. Hope this feature can be more used.

@lukaszgo1
Copy link
Contributor

IMO the main barrier to reviewing add-ons is the simple fact that they are done on GitHub. For a standard user this means they have to create an account on the website they will never use, which also is not in their mother tongue for many. In other cases i.e. Apple or Google stores the entire reviewing interface is localized, but you also have to create an account to use the store at all, so there is less intermediary steps before reviewing.

@nvdaes
Copy link
Sponsor Contributor

nvdaes commented Feb 16, 2024

Anyway, users that have GitHub accounts seem not to use this feature now, and Iwonder if creating a server infrastructure to review add-ons from the store should be prioritized if people who may review add-ons in the same platform where NVDA is developed are not leveraging this.
BTW, I'mlooking at the source code, and I don't understand why the review action and details are not presented in the installed add-ons tab. For me was logic thinking that just available add-ons should show the reviews action, but now that I'm thinking in this issue, I don't know what's roducing this, since I tried to implement this in a similar way respect the license URL, which is also optional.

@XLTechie
Copy link
Collaborator

XLTechie commented Feb 17, 2024 via email

@XLTechie
Copy link
Collaborator

XLTechie commented Feb 17, 2024 via email

@nvdaes
Copy link
Sponsor Contributor

nvdaes commented Feb 17, 2024

Hi Luke, first of all, hope you're well.
I'll read your suggested article. Personally, maybedue to my experience, I'm quite reluctant to leave bad reviews. Generally I'm not a person with a special fear to receive criticism from people that don't provide enough arguments, but I consider that add-ons are diverse, and people with low experience can participate to introduce themselves in NVDA's development, so I try to understand if someone seems to have submit an add-on to the store just for testing, like the add-on that I have in mind, which seems to have copied code from other add-on without too much criteria, just a piece and with dirty code, at least under my impression. I would leave a bad review since there is a better add-on, but please this is just an example and I looked at this months ago, and something may have changed from then, or I may have missed something. But you are probably right and bad reviews are more frequent.
In issue #15776 , Sean wrote:

Though, the best way to do this would be if NV Access creates our own server backend and reviews exist within the store.
However, this would be a major piece of work and is unlikely to be completed in the near future.

Also, GitHub is where NVDA is developed and even some tutorial has been shared about how to create issues, since users may want to report erros about NVDA. But I think that this feature hasn't been promoted.

@seanbudd seanbudd added p4 https://github.com/nvaccess/nvda/blob/master/projectDocs/issues/triage.md#priority quick fix good first issue github features these at https://github.com/nvaccess/nvda/contribute triaged Has been triaged, issue is waiting for implementation. labels Feb 19, 2024
@Nael-Sayegh
Copy link
Contributor

Hello,
I just read this issue.
I feel like 2 concepts are being mixed up, the act of revising an add-on's code, and the act for a user to give their feedback.
From what I understood, the function is rather intended for users to give their feedback.
I get the impression that @nvdaes is talking about his experience as a developer and not as a user.
Indeed, this function is not very used at the moment, for 2 reasons in my opinion:
-There hasn't been a publication about it on mailing lists, at least in France no one has talked about it
-The creation of a GitHub account which can pose a problem, but since GitHub allows you to connect with a Microsoft account, many people will be able to easily connect because the majority of people have a Microsoft account
Going back to the initial topic, I agree with @CyrilleB79 's suggestion to add the option in the tab for installed add-ons, I actually think it will make this option more popular.
And for external add-ons, it is just a matter of not displaying it for them.

@Adriani90
Copy link
Collaborator

@Nael-Sayegh Re external add-ons, it is not that trivial. In the installed add-ons tab, the user should actually know imediately when focusing an add-on whether there is a review page for it or not. I mean for people who have 30 add-ons installed, where 15 are external, they would have to open the actions menu for each of them to see if there is a review page available.
Maybe it makes sense to split the installed add-ons tab into two lists, one with reviewable add-ons, and one with add-ons without a review page.
Maybe called "submited add-ons list" and "external add-ons list".
Or even better, there could be a new filter combo box in the installed add-on tab called "source" where users can filter their list of add-ons between submited add-ons and external add-ons to find easier those with review page.

@CyrilleB79
Copy link
Collaborator Author

@Nael-Sayegh Re external add-ons, it is not that trivial. In the installed add-ons tab, the user should actually know imediately when focusing an add-on whether there is a review page for it or not. I mean for people who have 30 add-ons installed, where 15 are external, they would have to open the actions menu for each of them to see if there is a review page available.

If you want to read a review or leave a comment for a specific add-on, you select it in the list, open the context menu and check if there is a review item in this menu:

  • if yes, click it and read/write comment
  • if no, you will not be able to do anything; wether you are in the installed add-ons tab or in the available add-ons tab does not matter nor change anything.

@Adriani90, what is the problem?

@Nael-Sayegh
Copy link
Contributor

@Adriani90 I don't understand the problem you are talking about. Generally, the user knows whether the installed add-on is an external add-on or not, otherwise, the information is also indicated in the add-on's summary, so the user has all the elements to know the origin of the add-on. What do you think?

@Adriani90
Copy link
Collaborator

Adriani90 commented Apr 5, 2024 via email

@Adriani90
Copy link
Collaborator

Adriani90 commented Apr 5, 2024 via email

@Nael-Sayegh
Copy link
Contributor

@Adriani90 One can already filter add-ons by channel, in the list of channels one can choose among all/stable/dev/beta/external which allows for grouping the add-ons. Also, one can see the channel of an add-on in the list of installed add-ons directly like this: ObjectStudies, IA, IA2; Status: Enabled; Installed version: 2.1; Channel: External; Author: Pierre-Louis R This way one can see that the add-on is external and that there's no discussion to leave a review. Is this what you were talking about?

@Adriani90
Copy link
Collaborator

Adriani90 commented Apr 5, 2024 via email

@XLTechie
Copy link
Collaborator

XLTechie commented Apr 6, 2024 via email

@nvdaes
Copy link
Sponsor Contributor

nvdaes commented Apr 6, 2024

I think that we should exclude external add-ons from the review action, since add-ons in the store should have a unique name for the same add-on. Instead, an external add-on can have a name equal to other completely different add-on available in the store. If an author want to provide a page to review an external add-on, the add-on may provide a mechanism to do it even if it is not published in the store. What do you think, Luke?

@nvdaes
Copy link
Sponsor Contributor

nvdaes commented Apr 6, 2024

Hello, this is triaged. Seems I've fixed this locally. I'll create a PR for this. External add-ons will be excluded.

@XLTechie
Copy link
Collaborator

XLTechie commented Apr 6, 2024 via email

@nvdaes
Copy link
Sponsor Contributor

nvdaes commented Apr 6, 2024

Great @XLTechie . Then I'll proceed now with the PR to be tested by the community.

michaelDCurran pushed a commit that referenced this issue Apr 7, 2024
Fixes #16179

Summary of the issue:
The community reviews action, and the reviews URL, is not available in installed and incompatible add-ons tab of the store.

Description of user facing changes
The community reviews action, and the reviews URL, will be available in all tabs of the store.

Description of development approach
Fix reviewUrl variable assignment in the _createInstalledStoreModelFromData function of addonStore/models/addon.py
@nvaccessAuto nvaccessAuto modified the milestone: 2024.2 Apr 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue github features these at https://github.com/nvaccess/nvda/contribute p4 https://github.com/nvaccess/nvda/blob/master/projectDocs/issues/triage.md#priority quick fix triaged Has been triaged, issue is waiting for implementation.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

8 participants