Skip to content
This repository was archived by the owner on Jan 14, 2025. It is now read-only.

[ANDROID] support html tags for message and bigText fields #1469

Closed
wants to merge 3 commits into from
Closed

[ANDROID] support html tags for message and bigText fields #1469

wants to merge 3 commits into from

Conversation

wanderbon
Copy link

Support popular html tags in android notifications (without img)

@Dallas62 Dallas62 changed the base branch from master to dev June 8, 2020 10:13
@Dallas62 Dallas62 self-assigned this Jun 11, 2020
@Dallas62
Copy link
Collaborator

Hi @wanderbon
Thanks for this PR !
I will pick changes because I think there is some changes that are already applied and want to be sure there is no side effect.
This PR will be mentioned in the CHANGELOG after changes 😉

@vikasg603
Copy link

Hey, Any update on when this will get approved?

@Dallas62
Copy link
Collaborator

Dallas62 commented Feb 3, 2021

Hi,
Does iOS support the html format ?
I don't want to see this changes impact iOS device showing html tag if not parsed.

@wanderbon
Copy link
Author

Hi,
Does iOS support the html format ?
I don't want to see this changes impact iOS device showing html tag if not parsed.

Unfortunately, iOS does not support html in notifications

@silencer07
Copy link
Contributor

Hi! Can we ask the maintainer if we can merge this code. I need this functionality in the bigText field

@Dallas62
Copy link
Collaborator

Hi,
This PR will be merged only if it's applied to bigText.
message is shared with iOS and iOS doesn't support HTML.

@silencer07
Copy link
Contributor

@Dallas62 I will create a new PR that will satisfy your requirement. Will take inspiration from this code

@silencer07
Copy link
Contributor

New PR that takes inspiration from this PR. If the said PR is merged, we can safely discard this PR.

@wanderbon
Copy link
Author

This PR is closed because the associated PR is merged

@wanderbon wanderbon closed this Mar 17, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants