Skip to content

fix: handle ical.js group parameter form to prevent custom label corruption on save/re-fetch cycle#5378

Open
pallavibakale wants to merge 5 commits into
nextcloud:mainfrom
pallavibakale:fix/custom-label-for-contact-details-5302
Open

fix: handle ical.js group parameter form to prevent custom label corruption on save/re-fetch cycle#5378
pallavibakale wants to merge 5 commits into
nextcloud:mainfrom
pallavibakale:fix/custom-label-for-contact-details-5302

Conversation

@pallavibakale

Copy link
Copy Markdown
  • Fix custom label (X-ABLABEL) handling to correctly work with properties loaded from the server
  • Prevent prefix corruption (e.g NEXTCLOUD1.TEL) when re-applying a custom label after a save/fetch cycle
  • Fix propGroup and propLabel to detect the group parameter form that ical.js uses after parsing from server
image

Fixes: #5302

@hamza221 hamza221 added 2. developing Work in progress skill:frontend Issues and PRs that require JavaScript/Vue/styling development skills labels Jun 11, 2026

@hamza221 hamza221 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Hello,
Thank you for opening this PR,
The diff is really big and it looks like it's mostly spacing related fixes.
Can you please run npm ci then npm run lint:fix then push again

@codecov

codecov Bot commented Jun 11, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 0% with 34 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
...mponents/ContactDetails/ContactDetailsProperty.vue 0.00% 17 Missing ⚠️
src/mixins/PropertyMixin.js 0.00% 11 Missing and 6 partials ⚠️

📢 Thoughts on this report? Let us know!

…uption on save/re-fetch cycle

Signed-off-by: pallavibakale <pallavibakale09@gmail.com>
Signed-off-by: pallavibakale <pallavibakale09@gmail.com>
Signed-off-by: pallavibakale <pallavibakale09@gmail.com>
Signed-off-by: pallavibakale <pallavibakale09@gmail.com>
@pallavibakale pallavibakale force-pushed the fix/custom-label-for-contact-details-5302 branch from c71d75f to ccbed23 Compare June 11, 2026 15:26
@pallavibakale

Copy link
Copy Markdown
Author

Hello, Thank you for opening this PR, The diff is really big and it looks like it's mostly spacing related fixes. Can you please run npm ci then npm run lint:fix then push again

Hello Hamza, I have fixed the linting issues, the PR looks cleaner than before.
The DCO also failed, I ran the below to fix the issue:

git rebase HEAD~5 --signoff  
git push --force-with-lease origin fix/custom-label-for-contact-details-5302

Will do the same for the other PR as well.

@pallavibakale pallavibakale requested a review from hamza221 June 14, 2026 15:52
@DerDreschner DerDreschner added 3. to review Waiting for reviews and removed 2. developing Work in progress labels Jun 15, 2026
Signed-off-by: pallavibakale <pallavibakale09@gmail.com>
@pallavibakale pallavibakale force-pushed the fix/custom-label-for-contact-details-5302 branch from ccbed23 to 9613346 Compare June 15, 2026 15:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews skill:frontend Issues and PRs that require JavaScript/Vue/styling development skills

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Custom email/phone labels not working

3 participants