Skip to content
This repository has been archived by the owner on Apr 13, 2020. It is now read-only.

add time-saver note #66

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

yelizariev
Copy link
Contributor

No description provided.

Copy link
Member

@pedrobaeza pedrobaeza left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OK for me, although this is something outside Doodba

Copy link
Contributor

@yajo yajo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hmm I see you might be a little frustrated after losing a lot of time with that, isn't it?

I'd like to help you but TBH I don't think the scaffolding should be the place to write docs. If any, I prefer to put a link to the docs than writing them here. Don't you have a link to docs that say that? If you don't, it wouldn't be a problem to document this in doodba itself, as it affects the developer and doodba is the developer tool; then we could put a link here and even replace that short comment with something more elaborate.

OTOH I think this could be considered a bug and then you could open it to odoo, if they fix it we won't need this.

In any case, I don't think this is the place to document things, so I'd prefer to not merge this, at least as it is.

@pedrobaeza
Copy link
Member

That's not a bug, but a consequence of reading views directly from XML files.

@yajo
Copy link
Contributor

yajo commented Jan 17, 2020

If it worked in v9, it should be possible.

Also, the fact a view is loaded from the filesystem doesn't have anything to do with translations. When the view definition is stored in db, it's also stored in English, so they should be translatable.

@pedrobaeza
Copy link
Member

@yajo don't insist: that's something by design in Odoo and it's not a bug. It has been already confirmed multiple times.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants