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

[14.0] [ADD] fieldservice_portal #1146

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

Conversation

renda-dev
Copy link

Bridge module between fieldservice and portal that allows portal users to monitor work orders related to their locations.

@renda-dev renda-dev force-pushed the 14.0-add-fieldservice_order_portal branch from 518a3f7 to 1c5bdd3 Compare December 11, 2023 11:49
Copy link
Contributor

@aleuffre aleuffre left a comment

Choose a reason for hiding this comment

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

Code review, LGTM.

Suggest maybe renaming simply to fieldservice_portal?

@renda-dev renda-dev force-pushed the 14.0-add-fieldservice_order_portal branch from 1c5bdd3 to bf9f336 Compare January 9, 2024 14:22
@renda-dev renda-dev changed the title [14.0] [ADD] fieldservice_order_portal [14.0] [ADD] fieldservice_portal Jan 9, 2024
@renda-dev
Copy link
Author

Code review, LGTM.

Suggest maybe renaming simply to fieldservice_portal?

Done, thanks for the suggestion! @aleuffre

return request.render("fieldservice_portal.portal_my_fsm_orders", values)

@http.route(
["/my/workorder/<int:order_id>"],
Copy link
Sponsor Member

Choose a reason for hiding this comment

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

It's not better if it's fsm_order instead worker orders?

@max3903 max3903 added this to the 14.0 milestone Jan 31, 2024
@PicchiSeba PicchiSeba force-pushed the 14.0-add-fieldservice_order_portal branch from bf9f336 to 106a37e Compare March 4, 2024 11:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants