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] [FIX] maintenance_location_hr: Update maintenance_equipment.xml #383

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

Conversation

hazemha54
Copy link

Fix xml Error

Screenshot from 2024-02-20 14-17-10

Copy link
Member

@mamcode mamcode left a comment

Choose a reason for hiding this comment

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

Please can you give us more information on how the error shown in the screenshot occurs? I just can't understand why you want to add the fields employee_id and department_id if they are already present in the view.

@etobella
Copy link
Member

Mmmh. This isssue is probably related to the order of installation... If you increase the priority, that might be enough.

Copy link

github-actions bot commented Sep 1, 2024

There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days.
If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

@github-actions github-actions bot added the stale PR/Issue without recent activity, it'll be soon closed automatically. label Sep 1, 2024
@etobella etobella changed the title Update maintenance_equipment.xml [14.0] [FIX] maintenance_location_hr: Update maintenance_equipment.xml Sep 2, 2024
@etobella
Copy link
Member

etobella commented Sep 2, 2024

@hazemha54 can you change the commit name and port the change to next versions?

@github-actions github-actions bot removed the stale PR/Issue without recent activity, it'll be soon closed automatically. label Sep 8, 2024
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

Successfully merging this pull request may close these issues.

3 participants