joinmarket-webui / jam

Your sats. Your privacy. Your profit.
https://jamapp.org
MIT License
259 stars 53 forks source link

ui: remove twitter link from settings page #790

Closed 0xSaksham closed 3 months ago

0xSaksham commented 3 months ago

Resolves #750

theborakompanioni commented 3 months ago

Nice. Thanks @0xSaksham :pray:

Can you also remove the settings.jam_twitter from the language files?

0xSaksham commented 3 months ago

Sure man.

0xSaksham commented 3 months ago

Nice. Thanks @0xSaksham 🙏

Can you also remove the settings.jam_twitter from the language files?

Ready to Merge 👍

theborakompanioni commented 3 months ago

Nice. Thanks @0xSaksham 🙏 Can you also remove the settings.jam_twitter from the language files?

Ready to Merge 👍

Should it also be removed from the other language files?

0xSaksham commented 3 months ago

I didn't know that. I'll do a full codebase search and remove it from everywhere.

0xSaksham commented 3 months ago

Removed the Translation "jam_twitter" from files.

0xSaksham commented 3 months ago

@theborakompanioni Should I Squash and Merge this into main branch?