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

Location open link #4312

Closed
wants to merge 2 commits into from

Conversation

avinash-0007
Copy link

@avinash-0007 avinash-0007 commented Jun 22, 2022

This merge request allows to add a open link option when user enter a url in location field.

@tcitworld
Copy link
Member

tcitworld commented Jun 22, 2022

Location is a property to put physical addresses as plain text, not URLs.

The sames applies for the description field: #3233 (comment)

See the appropriate following issues #2895 and #1758 instead to use the correct property for this.

@avinash-0007
Copy link
Author

Location is a property to put physical addresses as plain text, not URLs.

The sames applies for the description field: #3233 (comment)

See the appropriate following issues #2895 and #1758 instead to use the correct property for this.

Location is a property to put physical addresses as plain text, not URLs.

The sames applies for the description field: #3233 (comment)

See the appropriate following issues #2895 and #1758 instead to use the correct property for this.

Location is a property to put physical addresses as plain text, not URLs.

The sames applies for the description field: #3233 (comment)

See the appropriate following issues #2895 and #1758 instead to use the correct property for this.

Yeah but if someone enter a url of a web meeting then there must be way to open it? so when you enter text it will not do anything but when you enter a url it give an option to open that link .

@tcitworld
Copy link
Member

Sure, but this need to use the other new property mentioned in the linked issue, not the location one.

@st3iny
Copy link
Member

st3iny commented Aug 8, 2023

Thank you for the contribution. This feature is now implemented via #5355.

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.

4 participants