[18.0][MIG] fastapi: Migration to 18.0 #471

cormaza · 2024-11-08T20:05:57Z

Change serialization error class to test retry failure

Depends on:

[18.0][MIG] endpoint_route_handler: Migration to 18.0 web-api#69

…pi service excution

* add default empty method to use as dependency to get the authenticated partner * improves the demo app to illustrate the way the dependency overrides mechanism can be used to provide the right implementation to use to retrieve the authenticated partner according to the security method configured on the app * add tests for the demo app to show how the TestClient class and the dependey overrides functianality should be used to easily write tests

This method can be used to get access to the fastapi.endpoint record into your router's methods

ensure transation is rolled back in case of error and allows override / extension of the error handling globally or by app

…ndlers

…thin fastapi apps

Co-authored-by: Fernando <fernandolachica@gmail.com>

…ted processing lang

A ribbon and a button are dislayed on the form view when a record needs to be synchronized. In the tree view the records to synchronize are displayed with the text decoration defined for warning. A button is also displayed on the line if sync is required and an server actions allows you to sync selected records in one click

In the same time, applies the security guidelines for the demo app

cormaza · 2024-11-19T16:44:47Z

Hi @cormaza, thanks for the work. Do you have any idea on how to fix the tests?

I'm not pretty sure, because if you can see test log, It seems every test works fine, but because log has an error text can't be processed as passed by CI, I think can be the parser version now aggregate another validation, maybe in older versions doesn't matter, We can ask to @pedrobaeza? WDYT?

pedrobaeza · 2024-11-19T16:47:07Z

I don't know about this repository. @lmignon may say.

marielejeune

LGTM (Code + Functional review)

lmignon · 2024-11-19T17:24:09Z

@cormaza the commit 4dd16b2 should solve this issue.

cormaza · 2024-11-19T18:16:12Z

@cormaza the commit 4dd16b2 should solve this issue.

Thank you, but doesn't work

cormaza · 2024-11-19T21:20:00Z

@lmignon I added in other test method, and now works, thank you for help

fastapi/tests/test_fastapi_demo.py

…ected

lmignon · 2025-01-30T13:15:45Z

/ocabot migration fastapi

lmignon

LGTM (waiting for the merge of the endpoint_route_handler addon)

dnplkndll

tested with demo api

OCA-git-bot · 2025-02-01T18:39:58Z

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

lmignon · 2025-02-03T08:09:43Z

/ocabot merge nobump

OCA-git-bot · 2025-02-03T08:09:50Z

This PR looks fantastic, let's merge it!
Prepared branch 18.0-ocabot-merge-pr-471-by-lmignon-bump-nobump, awaiting test results.

OCA-git-bot · 2025-02-03T08:13:58Z

Congratulations, your PR was merged at 7aad90c. Thanks a lot for contributing to OCA. ❤️

lmignon and others added 30 commits November 8, 2024 11:14

[ADD] Allows use of fastapi into Odoo!

119a1ec

[IMP] fastapi: Uses contextvars to hold and provide odoo env on fasta…

04d86c4

…pi service excution

[IMP] fastapi: renames fastapi.app into fastapi.endpoint

429da02

[IMP] fastapi: code cleanup

f8d8c43

[IMP] fastapi: Add dependency method

50cfd6c

This method can be used to get access to the fastapi.endpoint record into your router's methods

[IMP] fastapi: Implement error handling

ac0abb4

ensure transation is rolled back in case of error and allows override / extension of the error handling globally or by app

[IMP] fastapi: Allows to define the user executing the api calls

d8e1037

[IMP] fastapi: Add authenticated_partner_id into ir.rule eval context

7966964

[IMP] fastapi: Add helper code to ease development of search route ha…

38f7bd6

…ndlers

[IMP] fastapi: documentation

a16a53d

Fix a few typos

d58e092

Fix typo

e7ed75a

little fixes

56bbee1

[FIX] fastapi: Uses new implementation of endpoint_route_handler

2362418

[FIX] fastapi: change annotation to support python 3.7+

21113bf

[ADD] extendable_fastapi: New addon to allow the use of extendable wi…

f602324

…thin fastapi apps

[FIX] fastapi: typo

11f317e

Co-authored-by: Fernando <fernandolachica@gmail.com>

[IMP]fastapi: Uses the accept-languages header to determine the expex…

a00388c

…ted processing lang

[IMP] fastapi: Register only one handler by application

3b32e9e

[IMP] fastapi: Improves documentation

8546eff

[IMP] fastapi: Improves documentation on security aspects

59b6968

In the same time, applies the security guidelines for the demo app

[IMP] fastapi: Declared as Beta

47f0f73

[IMP] fastapi: Typos

39692dd

[UPD] Update fastapi.pot

01c3a2c

[UPD] README.rst

3d1e645

[DOC] fast_api: fix _get_fastapi_routers documentation

d9c6aa5

[DOC] fastapi: documentation fixes

5abd8c6

[UPD] README.rst

21b7581

marielejeune approved these changes Nov 19, 2024

View reviewed changes

cormaza force-pushed the 18.0-mig-fastapi branch from 3d12799 to 56ede08 Compare November 19, 2024 21:19

lmignon requested changes Nov 20, 2024

View reviewed changes

fastapi/tests/test_fastapi_demo.py Outdated Show resolved Hide resolved

cormaza added 2 commits November 20, 2024 09:56

[18.0][MIG] fastapi: Migration to 18.0

5353ef2

[PATCH] [FIX] fastapi: Disable exception logger when exception is exp…

d960109

…ected

cormaza force-pushed the 18.0-mig-fastapi branch from 56ede08 to d960109 Compare November 20, 2024 14:56

dnplkndll mentioned this pull request Jan 16, 2025

[18.0][MIG] fastapi_auth_jwt: Migration to 18.0 #490

Merged

2 tasks

OCA-git-bot added this to the 18.0 milestone Jan 30, 2025

OCA-git-bot mentioned this pull request Jan 30, 2025

Migration to version 18.0 #462

Open

18 tasks

lmignon reviewed Jan 30, 2025

View reviewed changes

lmignon approved these changes Jan 30, 2025

View reviewed changes

OCA-git-bot added the approved label Jan 30, 2025

[18.0][FIX] fastapi, use new authenticate signature and dict answer

4c718df

cormaza force-pushed the 18.0-mig-fastapi branch from 57944ca to 4c718df Compare January 30, 2025 15:11

dnplkndll approved these changes Feb 1, 2025

View reviewed changes

OCA-git-bot added the ready to merge label Feb 1, 2025

OCA-git-bot added the bot is merging ⏳ label Feb 3, 2025

OCA-git-bot merged commit 7a0af6a into OCA:18.0 Feb 3, 2025
7 checks passed

OCA-git-bot added merged 🎉 and removed bot is merging ⏳ labels Feb 3, 2025

Uh oh!

[18.0][MIG] fastapi: Migration to 18.0 #471

[18.0][MIG] fastapi: Migration to 18.0 #471

Uh oh!

Conversation

cormaza commented Nov 8, 2024 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

cormaza commented Nov 19, 2024

Uh oh!

pedrobaeza commented Nov 19, 2024

Uh oh!

marielejeune left a comment

Choose a reason for hiding this comment

Uh oh!

lmignon commented Nov 19, 2024

Uh oh!

cormaza commented Nov 19, 2024

Uh oh!

cormaza commented Nov 19, 2024

Uh oh!

Uh oh!

lmignon commented Jan 30, 2025

Uh oh!

lmignon left a comment

Choose a reason for hiding this comment

Uh oh!

dnplkndll left a comment

Choose a reason for hiding this comment

Uh oh!

OCA-git-bot commented Feb 1, 2025

Uh oh!

lmignon commented Feb 3, 2025

Uh oh!

OCA-git-bot commented Feb 3, 2025

Uh oh!

Uh oh!

OCA-git-bot commented Feb 3, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

17 participants

cormaza commented Nov 8, 2024 •

edited

Loading