Skip to content
This repository has been archived by the owner on Jun 7, 2023. It is now read-only.

Add payment browser notifications #52

Merged
merged 8 commits into from
Dec 19, 2019
Merged

Conversation

rihardsgravis
Copy link
Contributor

If supported by the browser, show notifications for incoming and outgoing transactions .

@rihardsgravis rihardsgravis added Feature New feature or request Browser Browser and PWA application related issues UI/UX User Interface related issues labels Dec 13, 2019
@rihardsgravis rihardsgravis self-assigned this Dec 13, 2019
@rihardsgravis rihardsgravis changed the title Add payment browser notification Add payment browser notifications Dec 13, 2019
@nothingismagick
Copy link
Contributor

Somebody brought up native notifications at tauri yesterday:

tauri-apps/tauri#155 (comment)

@rihardsgravis
Copy link
Contributor Author

Awesome @nothingismagick! I was also looking into https://docs.rs/notify/5.0.0-pre.1/notify/ as an option for the desktop version. But if it's added to Tauri core - even better 🎉

@nothingismagick
Copy link
Contributor

We're working hard to launch the alpha this month - just one last tricky bit to solve. Its good to know that spark could consume it - so we'd have a smoke-test basically ready to go.

Rihards Gravis added 4 commits December 18, 2019 10:37
* master:
  chore: update @iota/account package to 1.0.0-beta.25
  Fix locale typo
  Switch to Typescript tests
  - Update layouts max-width - Limit Amount input minimum value to 0 - Add transaction input link hover color - Update SeedVault minimum password length to 8 chars
  fix: update "choose password" description
  fix: do not attempt to destroy scanner object if it has already been destroyed
  chore: remove unnecessary log
  fix: make sure outgoing transactions are displayed under "Sent" tab
  fix: check for null values before accessing marketPrice.currency prop
  Add missing request amount checks
  Add an error message if address generation fails
  chore(readme): use yarn not npm
  chore: bump @iota packages
  Add missing helpers tests
# Conflicts:
#	src-ui/lib/account.ts
@rihardsgravis rihardsgravis merged commit a3bda3c into master Dec 19, 2019
@rihardsgravis rihardsgravis deleted the feat/notifications branch December 19, 2019 15:45
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Browser Browser and PWA application related issues Feature New feature or request UI/UX User Interface related issues
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants