add separate screen for amount confirmations #15

Merged
anomalist merged 13 commits from separate-dbs into master 2 years ago
Owner

Well, this is now a significant one:

  • It has the separate screen for seeing all given items and for confirming them.

  • It also has the ability to switch between accounts. Just go to /import-account and import another one and you can switch between them on the /account screen.

I plan to do one more round of tests tomorrow, and maybe some small tweaks before I'm ready for prime time, but this is ready for review.

Well, this is now a significant one: * It has the separate screen for seeing all given items and for confirming them. * It also has the ability to switch between accounts. Just go to /import-account and import another one and you can switch between them on the /account screen. I plan to do one more round of tests tomorrow, and maybe some small tweaks before I'm ready for prime time, but this is ready for review.
trentlarson added 2 commits 2 years ago
trentlarson reviewed 2 years ago
@Options({})
export default class ContactsView extends Vue {
contact: Contact | null = null;
Poster
Owner

At first I tried this:

contact?: Contact;

... but it complained at me in the console:

[Vue warn]: Property "contactName" was accessed during render but is not defined on instance.

... and it wouldn't load my information -- even though the "contacts.get" a few lines lower loads the right entry, the rest of the page wouldn't recognize that I set this.contact (or any other this value)... it was like the page was broken.

So... this works, but it doesn't seem clean, and if there's a better way than I'm all ears.

At first I tried this: `contact?: Contact;` ... but it complained at me in the console: `[Vue warn]: Property "contactName" was accessed during render but is not defined on instance. ` ... and it wouldn't load my information -- even though the "contacts.get" a few lines lower loads the right entry, the rest of the page wouldn't recognize that I set `this.contact` (or any other `this` value)... it was like the page was broken. So... this works, but it doesn't seem clean, and if there's a better way than I'm all ears.
Poster
Owner

Have you tried:

contact: Nullable<Contact>;

?

Have you tried: `contact: Nullable<Contact>;` ?
Poster
Owner

Both lint & Web Storm say that is not defined

Both lint & Web Storm say that is not defined
trentlarson added 1 commit 2 years ago
trentlarson added 2 commits 2 years ago
trentlarson added 1 commit 2 years ago
trentlarson added 1 commit 2 years ago
trentlarson added 1 commit 2 years ago
trentlarson added 1 commit 2 years ago
trentlarson added 2 commits 2 years ago
trentlarson added 1 commit 2 years ago
trentlarson changed title from WIP: add separate screen for amount confirmations to add separate screen for amount confirmations 2 years ago
anomalist added 1 commit 2 years ago
anomalist merged commit 1d47a90836 into master 2 years ago
The pull request has been merged as 1d47a90836.
Sign in to join this conversation.
No reviewers
No Label
No Milestone
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.

Loading…
There is no content yet.