Skip to content

Prepare release of version 1.8.0#1649

Merged
Alkarex merged 1 commit intoFreshRSS:devfrom
Alkarex:release-1.8.0
Oct 1, 2017
Merged

Prepare release of version 1.8.0#1649
Alkarex merged 1 commit intoFreshRSS:devfrom
Alkarex:release-1.8.0

Conversation

@Alkarex
Copy link
Copy Markdown
Member

@Alkarex Alkarex commented Sep 30, 2017

@Alkarex Alkarex added this to the 1.8.0 milestone Sep 30, 2017
@Alkarex
Copy link
Copy Markdown
Member Author

Alkarex commented Sep 30, 2017

This release is long due, with several important bug fixes. It was initially planned to be a minor release 1.7.1, but since we have had a lot of code changed for Travis, and a change of minimal PHP version, I think it is better to go for 1.8.0.

Changelog: https://github.com/FreshRSS/FreshRSS/blob/dev/CHANGELOG.md

  • Compatibility:
    • Minimal PHP version increased to PHP 5.3.8+ to fix sanitize bug #1604
    • Add support for PHP 7.1 in the API #1584, #1594
  • UI
    • New page for subscription tools #1534
    • Adjustments to the padding of the tree of categories and feeds 1589
    • Fix feed column position after lazy-loading images #1616
    • Force UI controls for HTML5 video and audio #1642
    • Fix share menu on small screens #1645
    • Go back to previous view when collapsing article #1177
  • CLI
    • New command ./cli/update-user to update user settings #1600
  • I18n
  • Bug fixing
    • Fix API compatibility bug between PostgreSQL and EasyRSS #1603
    • Fix PostgreSQL error when adding entries with duplicated GUID #1610, #1614
    • Fix for RSS feeds containing HTML in author field #1590
    • Fix logout issue in global view due to CSRF #1591
  • Misc.
    • Travis continuous integration #1619
    • Allow longer database usernames #1597

Thanks to all contributors: @aledeg @marienfressinaud @Frenzie @Alwaysin @gsongsong @romibi @sandfoxme @Trim @rcutanda @araujo0205 ...

If you have a chance to give a try to the /dev version before tomorrow evening, please post some feedback here :-)

@GeoZzNL @redinsight @giorgiobeggiora @geekafy @oupala @NerosTie @mathieumd @subic @auricom ...

@Alkarex Alkarex mentioned this pull request Sep 30, 2017
@araujo0205
Copy link
Copy Markdown
Contributor

I tested it and everything is working.
The language is Portuguese (Brazil) not Portuguese (Brazilian). Brazilian is the capital of Brazil :)

@Frenzie
Copy link
Copy Markdown
Member

Frenzie commented Oct 1, 2017

Seems fine to me. 👍

@araujo0205 It's Brazilian (the adjective) Portuguese. Whether it's right or wrong depends on whether you want to write Dutch (Belgian)/German (Austrian)/Portuguese (Brazilian) or Dutch (Belgium)/German (Austria)/Portuguese (Brazil). I think it's completely arbitrary, but in the language dropdown list the system should be internally consistent. :-)

@Alkarex
Copy link
Copy Markdown
Member Author

Alkarex commented Oct 1, 2017

Thanks for the feedback @araujo0205 and @Frenzie 👍
For the record, the capital of Brazil is Brasília without 'n' :-)

@aledeg
Copy link
Copy Markdown
Member

aledeg commented Oct 1, 2017

I didn't tested it thoroughly but so far it looks good to me.

@araujo0205
Copy link
Copy Markdown
Contributor

Thanks @Alkarex and @Frenzie . The internationalization is new for me.
I thought there was change depending on the language. Like Brazil and Brazil :)

@Alkarex Alkarex merged commit f241fc1 into FreshRSS:dev Oct 1, 2017
@Alkarex Alkarex deleted the release-1.8.0 branch October 1, 2017 16:22
@NerosTie
Copy link
Copy Markdown

NerosTie commented Oct 2, 2017

Still no new update available in my freshrss, is it normal?

@aledeg
Copy link
Copy Markdown
Member

aledeg commented Oct 2, 2017

Yes. See here

@marienfressinaud
Copy link
Copy Markdown
Member

Update should be ok now :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants