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

Adjust Makefile, Changelog and more #2390

Merged
merged 2 commits into from
Oct 24, 2023
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion .github/dependabot.yml
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ version: 2
updates:
# Maintain dependencies for npm
- package-ecosystem: "npm"
directory: "/js"
directory: "/"
schedule:
interval: "daily"
labels:
Expand Down
7 changes: 7 additions & 0 deletions AUTHORS.md
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@
* [Benjamin Brahmer](mailto:[email protected])
* [Alessandro Cosentino](mailto:[email protected])
* [Marco Nassabain](mailto:[email protected])
* [Devlin Junker](mailto:[email protected])
* [Robin Appelman](mailto:[email protected])
* [Sean Molenaar](mailto:[email protected])
* [Gregor Tätzner](mailto:[email protected])
Expand Down Expand Up @@ -37,6 +38,8 @@
* [Thomas Tanghus](mailto:[email protected])
* [Volkan Gezer](mailto:[email protected])
* [Xéfir Destiny](mailto:[email protected])
* [devlinjunker](mailto:[email protected])
* [hydrian](mailto:[email protected])
* [Daniel Opitz](mailto:[email protected])
* [Daniel Rheinbay](mailto:[email protected])
* [Lars Bensmann](mailto:[email protected])
Expand All @@ -49,6 +52,7 @@
* [Christoph Stenglein](mailto:[email protected])
* [Daniel Kesselberg](mailto:[email protected])
* [ELHADDAD Hamza](mailto:[email protected])
* [IgorA100](mailto:[email protected])
* [Jakob Sack](mailto:[email protected])
* [ManuelSailer](mailto:[email protected])
* [Qingping Hou](mailto:[email protected])
Expand Down Expand Up @@ -127,10 +131,12 @@
* [Freddo](mailto:[email protected])
* [Freddo3000](mailto:[email protected])
* [Greg](mailto:[email protected])
* [HK2FB](mailto:[email protected])
* [Hagen](mailto:[email protected])
* [Hanzei](mailto:[email protected])
* [Hendrik Leppelsack](mailto:[email protected])
* [Jasper Knockaert](mailto:[email protected])
* [Jimmy Boucher](mailto:[email protected])
* [Kevin Decherf](mailto:[email protected])
* [Kuba Orlik](mailto:[email protected])
* [Maceček Richard](mailto:[email protected])
Expand All @@ -142,6 +148,7 @@
* [Michael Grosser](mailto:[email protected])
* [Michael Hamann](mailto:[email protected])
* [Michael Holley](mailto:[email protected])
* [Mynacol](mailto:[email protected])
* [NanoSector](mailto:[email protected])
* [Nick Frey](mailto:[email protected])
* [Oliver Herst](mailto:[email protected])
Expand Down
11 changes: 5 additions & 6 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -5,12 +5,11 @@ The format is mostly based on [Keep a Changelog](https://keepachangelog.com/en/1
# Unreleased
## [25.x.x]
### Changed
- Vue Rewrite
# Releases
## [24.x.x]
### Changed
- Set User Agent for curl in Scraper

- Major Rewrite of the Frontend with Vue JS (#748)
For comments and suggestions for the new UI, please use this: https://github.com/nextcloud/news/discussions/2388
- Set User Agent for curl in Scraper (#2380)
- Drop support for Nextcloud 26, Supported 27

### Fixed

# Releases
Expand Down
9 changes: 4 additions & 5 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -92,10 +92,10 @@ else
@exit 1
endif

# Removes the appstore build
# Removes the appstore build and compiled js files
.PHONY: clean
clean:
rm -rf ./build
rm -rf ./build ./js/*

# Reports PHP codestyle violations
.PHONY: phpcs
Expand Down Expand Up @@ -171,9 +171,8 @@ appstore:
find $(appstore_sign_dir) -name .htaccess -exec rm {} \;

# on macOS there is no option "--parents" for the "cp" command
mkdir -p $(appstore_sign_dir)/$(app_name)/js/build $(appstore_sign_dir)/$(app_name)/js/admin
cp js/build/app.min.js $(appstore_sign_dir)/$(app_name)/js/build
cp js/nextcloud-news-admin-settings.js* $(appstore_sign_dir)/$(app_name)/js/build
mkdir -p $(appstore_sign_dir)/$(app_name)/js
cp js/* $(appstore_sign_dir)/$(app_name)/js/

# export the key and cert to a file
@if [ ! -f $(cert_dir)/$(app_name).key ] || [ ! -f $(cert_dir)/$(app_name).crt ]; then \
Expand Down
3 changes: 0 additions & 3 deletions README.md
Original file line number Diff line number Diff line change
@@ -1,7 +1,4 @@
# Nextcloud News app

**We need help with the frontend, check the issue tracker if you are interested!**

![Release status](https://github.com/nextcloud/news/workflows/Build%20and%20publish%20app%20release/badge.svg) ![Integration Tests](https://github.com/nextcloud/news/workflows/Integration%20Tests/badge.svg) ![Frontend tests](https://github.com/nextcloud/news/workflows/Frontend%20tests/badge.svg) [![Code coverage](https://img.shields.io/codecov/c/github/nextcloud/news.svg?style=flat)](https://codecov.io/gh/nextcloud/news/)

The News app is an RSS/Atom feed aggregator. It offers a [RESTful API](https://nextcloud.github.io/news/developer/#apis) for app developers. The source code is [available on GitHub](https://github.com/nextcloud/news)
Expand Down
2 changes: 1 addition & 1 deletion appinfo/info.xml
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@ Create a [feature request](https://github.com/nextcloud/news/discussions/new)

Report a [feed issue](https://github.com/nextcloud/news/discussions/new)
]]></description>
<version>24.0.0</version>
<version>25.0.0-alpha1</version>
<licence>agpl</licence>
<author>Benjamin Brahmer</author>
<author>Sean Molenaar</author>
Expand Down
1 change: 0 additions & 1 deletion lib/Controller/PageController.php
Original file line number Diff line number Diff line change
Expand Up @@ -96,7 +96,6 @@ public function index(): TemplateResponse
$csp->addAllowedImageDomain('*')
->addAllowedMediaDomain('*')
->addAllowedConnectDomain('*')// chrome breaks on audio elements
->allowEvalScript(true)
->addAllowedFrameDomain('https://youtube.com')
->addAllowedFrameDomain('https://www.youtube.com')
->addAllowedFrameDomain('https://player.vimeo.com')
Expand Down
2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
{
"name": "nextcloud-news",
"description": "An RSS/Atom feed reader",
"main": "js/app.min.js",
"main": "js/nextcloud-news-main.js",
"scripts": {
"prebuild": "npm install && npm ci",
"build": "NODE_ENV=production webpack --progress --config webpack.js",
Expand Down