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

Saving list overwrites namespace_id with 0 #13

Closed
worldworm opened this issue Aug 23, 2022 · 9 comments
Closed

Saving list overwrites namespace_id with 0 #13

worldworm opened this issue Aug 23, 2022 · 9 comments

Comments

@worldworm
Copy link

Hello there,
when you edit and save a list in the app, the namespace_id is set to 0. This makes the list disappear in the web frontend and in the app. You need to manually send a post request and reset the namespace back to the original one.
I'm not sure if it's a bug in the app or the api. If it is the api just give me a heads-up and I will open an issue in Gitea 😀

Tested with app version v0.0.19-alpha

@k9withabone
Copy link
Contributor

@Benimautner Maybe this is something to do with the switch to HttpClient (6f43c93) again?

@Benimautner
Copy link
Collaborator

Can't reproduce this. What API version are you using?

@worldworm
Copy link
Author

I'am using API-Version: v0.19.2
I will try it tonight with the demo instance to rule out my setup.

@Benimautner
Copy link
Collaborator

Even though I don't think this has to do with the switch to HttpClient as @k9withabone suggested, we have switched back. If you don't mind, please give the file attached to this comment a try.

@worldworm
Copy link
Author

The problem also occurs when using the demo instance at https://try.vikunja.io/.
The custom version of @Benimautner also has this problem.

Record_2022-08-25-23-10-36.mp4

@Benimautner
Copy link
Collaborator

I'm using API version v0.18.1 on my prod release. Everything works flawlessly there.

I just updated my dev release to v0.19.2 and am now experiencing the same bug.

@kolaente I read the docs, there seems to be a namespace_id when updating a list. I am guessing this is mandatory now and through the absence of it the list gets saved to 0?
I will include the namespace id and check if it's still happening.

@Benimautner
Copy link
Collaborator

fixed in v0.0.20-alpha
was indeed the missing namespace_id. is this intended behavior @kolaente ?

@kolaente
Copy link
Member

kolaente commented Sep 1, 2022

is this intended behavior @kolaente ?

@Benimautner Kind of. The frontend always sends the namespace_id when upgrading a list but it should not be possible to set it to 0. I'll fix this in the api.

@kolaente
Copy link
Member

kolaente commented Oct 1, 2022

Now also fixed in the api.

vikunja-bot pushed a commit to go-vikunja/vikunja that referenced this issue Oct 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants