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

Call for testing /dev before 0.9.2 beta #678

Closed
marienfressinaud opened this issue Oct 24, 2014 · 30 comments
Closed

Call for testing /dev before 0.9.2 beta #678

marienfressinaud opened this issue Oct 24, 2014 · 30 comments

Comments

@marienfressinaud
Copy link
Member

I had planned a new version for the next week (31th) so it's time to test the dev branch! With my last commits there are a lot of features which can be broken (but not so much I think :))

I18n fixes (see #655 and #648) will be delayed for the next beta since there are a lot of strings to change. i18n.freshrss.org will need some improvements to improve security and features.

Hope all will be ok and don't hesitate to open new tickets and mentioning this one if you think your problem should be corrected in the coming beta.

@Draky50110
Copy link

Updating user profile : when saving, little notif (upscreen) is... a string like " users.profil.updated ❌ " instead of a "normal sentence".

@marienfressinaud
Copy link
Member Author

It is known for untranslated strings, I wanted to fix that today or tomorrow :)

@Alwaysin
Copy link
Contributor

"You lack PDO for SQLite (it's ok if PDO is good)." in red even if we use MySQL. Maybe it can be a little confusing?

Also, why not merge "statistics", "logs" and "about" with "configuration" and "administration"? I mean just adding a third category ("miscellaneous"?) so that we can switch more easily between pages.

@Alwaysin
Copy link
Contributor

"Dancing titles" are back : #534 (but only in english language, and not in french ??)

@Draky50110
Copy link

In checking, "Table des catégories mal configurée" : what do I need to do ?

@marienfressinaud
Copy link
Member Author

"You lack PDO for SQLite (it's ok if PDO is good)." in red even if we use MySQL. Maybe it can be a little confusing?

True, I have to fix that. Thanks!

Also, why not merge "statistics", "logs" and "about" with "configuration" and "administration"?

Maybe… I'm not sure… "statistics", "logs" and "about" are only visualisation of information about FreshRSS, "configuration" and "administration" are actions. But I will consider it ;)

"Dancing titles" are back : #534 (but only in english language, and not in french ??)

I can't reproduce it and I don't think it's linked to my last changes. Moreover, there is no reason it happens in English and not in French. Really strange :s

In checking, "Table des catégories mal configurée" : what do I need to do ?

I think it's an additional column (color) which has been removed in a previous version but you have still it. There is no real problem. Later, there will be an option to fix the table (not this version). If you think it's problematic to have no information about the problem, I may remove this feature from the next bêta and waiting for the fixing feature?

@Alwaysin
Copy link
Contributor

Dancing titles appear only with Chrome and IE (last version each, no cache, etc), and only in english ; in Firefox it works fine.

marienfressinaud added a commit that referenced this issue Oct 26, 2014
- Fix i18n strings
- Fix typo profil -> profile

See #678
marienfressinaud added a commit that referenced this issue Oct 26, 2014
Show only one message for both mysql and sqlite pdo conf. If one of them is ok,
PDO is ok.

See #678
@marienfressinaud
Copy link
Member Author

I have fixed strings and there is only one message for PDO (no more "it's ok if PDO is good").

@Alwaysin > for the menu I don't touch anything but later I think statistic item will move in another place (maybe at the right of "Manage subscriptions" button). Logs will move in one of administration or configuration category and "About" will not change.

I look for dancing titles now ;)

@marienfressinaud
Copy link
Member Author

I understand the dancing title bug but I'm not sure I want to fix it for the moment: it's related to the date item. In english we add a <sup>th</sup> element which is in conflict with the .flux:not(.current):hover .item.title rule. I wanted to try another idea on .flux_header (display: flex;) but it has to wait at least next version.

@marienfressinaud
Copy link
Member Author

aside_feed is a bit larger now, let me know if it's too much or if it's ok.

@Draky50110
Copy link

@marienfressinaud maybe a more detailed message for the checking like "click here to see the error message" to see the detailed output ?
If not important, as it's "dev", not important to fix as it is no a block to the FRSS use ;)

@aledeg
Copy link
Member

aledeg commented Oct 26, 2014

So far, nothing special to mention except the excellent work.

@Alwaysin
Copy link
Contributor

Thanks for the string, and thanks for the answer on the menu : it looks good for me if you do this :)

I'm not a fan of the larger aside_feed, but it doesn't bother me more than this. It's just that it causes issues with the arrows at the bottom :

2014-10-26 17_28_13-private feed for alwaysin freshrss

marienfressinaud added a commit that referenced this issue Oct 26, 2014
@marienfressinaud
Copy link
Member Author

@Draky50110 > for the moment there is no error message. I think I will remove the database checkup for the beta and add it back when fixing db will be available.

@aledeg > thanks! :)

@Alwaysin > I have fixed the size of menu but I'm still not sure it's a good idea. At least it's better on Epiphany because subscription button was overflowing the aside before.

Note you can add a custom css file as an extension (system will be improved later):

  1. Create the file ./extensions/xCustomCSS/style.css
  2. Add your CSS rules in it. For instance:
.aside {
    width: 250px;
}
#nav_entries {
    width: 250px;
}

to get the previous width back.

@Alwaysin
Copy link
Contributor

Thanks for the tip !! 👍

@Alkarex
Copy link
Member

Alkarex commented Oct 26, 2014

If I am not mistaken, the "adjust showing" (adapter l'affichage) is not working anymore: it shows no article, when all my articles are read instead of showing the read ones.

@Draky50110
Copy link

t works for me but you need adjust + do not check "hide cat" and try to play with icons on main creen (2 firsts : read + unread).
Took me some tests to see the good result.

@Alwaysin
Copy link
Contributor

@marienfressinaud désolé ce n'est pas trop le sujet, mais j'ai mis l'extension comme tu me l'as conseillé et en vue mobile les catégories restent et refusent de partir, ce qui fait que c'est inutilisable :(
Un moyen rapide de corriger ou faut faire la modif dans la source ?

@Alwaysin
Copy link
Contributor

Est-ce que l'accès API fonctionne chez vous ? Je n'arrive plus à me connecter avec News+, mais je ne sais pas si c'est une erreur de mon côté.

Dans api.log j'ai :

User =>
clientLogin(Alwaysin)
unauthorized()

Alors que j'entre bien le mot de passe que j'utilise pour me logguer sur l'interface web et que " Autoriser l’accès par API (nécessaire pour les applis mobiles)" est coché.

@Alkarex
Copy link
Member

Alkarex commented Oct 28, 2014

J'ai eu un problème un peu similaire. Essaye en décochant l'accès API, puis
en le cochant à nouveau, et éventuellement en resaisissant le mot de passe
API.

@Alwaysin
Copy link
Contributor

J'ai déjà essayé en décochant, validant, cochant à nouveau et validant à nouveau et ça n'a pas changé :(
Le mot de passe API il n'est plus là, si ? Je ne le trouve plus et donc je pensais qu'il avait été fusionné avec le mot de passe de connexion "normal".

@Alkarex
Copy link
Member

Alkarex commented Oct 28, 2014

Si si, le mot de passe API est toujours là, dans la page profil, mais
requiert que l'accès API soit activé pour être visible.

@Alwaysin
Copy link
Contributor

Ok super merci, je n'avais pas vu le champ apparaître après la re-validation du paramètre API. Ca fonctionne maintenant :D

@marienfressinaud
Copy link
Member Author

Ce qui est étrange là-dedans c'est qu'a priori je n'ai pas touché à la partie API… menfin tant que ça marche c'est le principal :)

marienfressinaud added a commit that referenced this issue Oct 31, 2014
@marienfressinaud
Copy link
Member Author

I think we're good for the release :)

@aledeg
Copy link
Member

aledeg commented Oct 31, 2014

I am looking forward to update my server :)

@marienfressinaud
Copy link
Member Author

Done! ee50df5

Let me know if anything goes wrong!

@marienfressinaud
Copy link
Member Author

Oops! I have found some bugs :-/ Try to fix them quickly!

@marienfressinaud
Copy link
Member Author

Patched!

@Alkarex
Copy link
Member

Alkarex commented Nov 5, 2014

Feedback about performance:

  • I run on a modest server: OVH Kimsufi KS-1, Atom, 2GB memory

ks-1

  • Linux Ubuntu 14.04.1 LTS, Apache 2.4.7 (over HTTPS), PHP 5.5.9, MySQL 5.5.40
  • I use the current FreshRSS 0.10-dev, have 230k+ articles (300MB+ database)

archivage

  • Requests are most of the time < 200ms

performances

@Alkarex Alkarex removed the Work in progress 🚧 Wait before merging label Nov 5, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants