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

[NC28, News 25.X] Updating fields in admin settings not possible #2533

Closed
2 tasks done
Grotax opened this issue Jan 4, 2024 · 3 comments
Closed
2 tasks done

[NC28, News 25.X] Updating fields in admin settings not possible #2533

Grotax opened this issue Jan 4, 2024 · 3 comments
Labels
bug frontend impact Javascript/Frontend code

Comments

@Grotax
Copy link
Member

Grotax commented Jan 4, 2024

IMPORTANT

Read and tick the following checkbox after you have created the issue or place an x inside the brackets ;)

  • I have read the CONTRIBUTING.md and followed the provided tips
  • I accept that the issue will be closed without comment if I do not check here
  • [x I accept that the issue will be closed without comment if I do not fill out all items in the issue template.

Explain the Problem

What problem did you encounter?

Steps to Reproduce

Explain what you did to encounter the issue

  1. Open Admin Settings for news
  2. Change any value
  3. Check browser console and no update notification on screen

System Information

  • News app version: 25.0.0-alpha3
  • Nextcloud version: 28.0.1
  • Cron type: (system cron/python updater/...)
  • PHP version: --
  • Database and version:--
  • Browser and version: Firefox 121.0
  • OS and version: Manjaro Linux XFCE
Contents of nextcloud/data/nextcloud.log
Paste output here
Contents of Browser Error Console Read http://ggnome.com/wiki/Using_The_Browser_Error_Console if you are unsure what to put here
TypeError: this.$refs.field is undefined
    mounted index.mjs:35
    VueJS 9
    Wt</<.on["update:value"]< AdminSettings.vue:13
    VueJS 4
    handleInput NcTextField.js:2
    VueJS 51
    <anonymous> main-admin.js:20
    <anonymous> main-admin.js:20
    <anonymous> main-admin.js:20
vue.runtime.esm.js:3049:16

@Grotax Grotax added bug frontend impact Javascript/Frontend code labels Jan 4, 2024
@Grotax
Copy link
Member Author

Grotax commented Jan 4, 2024

From my first checks we might need to drop support for NC27 and upgrade the frontend libs according to Nextcloud 28.

@mhzawadi

This comment has been minimized.

@z3ntu
Copy link

z3ntu commented Jun 6, 2024

As workaround for changing settings the command line seems to be usable:

$ occ config:list news
{
    "apps": {
        "news": {
            "types": "",
            "enabled": "yes",
            "autoPurgeMinimumInterval": "60",
            "autoPurgeCount": "10000",
            "maxRedirects": "10",
            "feedFetcherTimeout": "60",
            "exploreUrl": "",
            "updateInterval": "3600",
            "useCronUpdates": "",
            "installed_version": "25.0.0-alpha6"
        }
    }
}
$ occ config:app:set --value 1 news useCronUpdates
Config value 'useCronUpdates' for app 'news' is now set to '1', stored as mixed in fast cache

Grotax added a commit that referenced this issue Oct 3, 2024
Fixed
- Use updated user agent when fetching feeds and favicons (#2788)
- Allow feed title to be null in DB. (#2745)
- Store HTTP last modified date from response header (#2724)
- Admin settings could not be saved (#2533)

Signed-off-by: Benjamin Brahmer <info@b-brahmer.de>
Grotax added a commit that referenced this issue Oct 3, 2024
Fixed
- Use updated user agent when fetching feeds and favicons (#2788)
- Allow feed title to be null in DB. (#2745)
- Store HTTP last modified date from response header (#2724)
- Admin settings could not be saved (#2533)

Signed-off-by: Benjamin Brahmer <info@b-brahmer.de>
Grotax added a commit that referenced this issue Oct 3, 2024
Fixed
- Use updated user agent when fetching feeds and favicons (#2788)
- Allow feed title to be null in DB. (#2745)
- Store HTTP last modified date from response header (#2724)
- Admin settings could not be saved (#2533)

Signed-off-by: Benjamin Brahmer <info@b-brahmer.de>
@Grotax Grotax closed this as completed Oct 3, 2024
Grotax added a commit that referenced this issue Oct 4, 2024
Fixed
- Use updated user agent when fetching feeds and favicons (#2788)
- Allow feed title to be null in DB. (#2745)
- Store HTTP last modified date from response header (#2724)
- Admin settings could not be saved (#2533)

Signed-off-by: Benjamin Brahmer <info@b-brahmer.de>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug frontend impact Javascript/Frontend code
Projects
None yet
Development

No branches or pull requests

3 participants