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

Currency Rate Update - Next Run - in trecut #76

Open
dhongu opened this issue Mar 15, 2015 · 3 comments
Open

Currency Rate Update - Next Run - in trecut #76

dhongu opened this issue Mar 15, 2015 · 3 comments

Comments

@dhongu
Copy link
Contributor

dhongu commented Mar 15, 2015

Daca din diverse motive serverul nu functioneaza intr-o zi campul next_run din Currency_rate_update_service va avea data din trecut. Aceasta duce la blocarea actualizarii cursului valutar.

    @api.multi
    def run_currency_update(self):
        # Update currency at the given frequence
        services = self.search([('next_run', '=', fields.Date.today())])
        services.with_context(cron=True).refresh_currency()

Nu ar fi bine daca in metoda run_currency_update in loc de conditia '=' sa fie '<=' ?

@feketemihai
Copy link
Owner

Pentru a nu strica perioada cred ca ar trebui sa scriem next_run chiar si fara actualizare...in caz de actualizare saptamanala, lunara nu o sa schimbe data in care se actualizeaza...o sa faca actualizarea in data stabilita cu termenul stabilit...Ce ziceti?

@feketemihai
Copy link
Owner

Scuze...e ok si cu <= ca scrie urmatorul in functie de next run...devi va pastra perioada...

@feketemihai
Copy link
Owner

Ce ziceti renuntam la modul din l10n-romania si il folosim pe cel de pe OCA??? Si sa facem direct acolo Issue???
@dhongu @yoyo2k @alexteodor

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

2 participants