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

Bug: Adding Holdings not possible (Investment / Brokerage Account) #1657

Open
dmnk opened this issue Jan 23, 2025 · 4 comments
Open

Bug: Adding Holdings not possible (Investment / Brokerage Account) #1657

dmnk opened this issue Jan 23, 2025 · 4 comments
Labels
ℹ️ Needs info Further information is requested

Comments

@dmnk
Copy link

dmnk commented Jan 23, 2025

Describe the bug
The form to add a holding doesn't "hold" the Ticker Symbol. Maybe the validation of the symbol is broken?

To Reproduce
Steps to reproduce the behavior:

  1. create a brokerage account
  2. click new holding
  3. enter required details for the holding (btw, the Ticker-Symbol isn't marked as required)
  4. after leaving the field, it disposes the entered string, and again shows the grayed out placeholder AAPL

Expected behavior
Keep the Ticker Symbol; maybe present an info/popup warning if the current value cannot be fetched and the user is expected to keep that up to date.

What version of Maybe are you using?
Self-Hosted 0.3.0

What operating system and browser are you using?
The more info the better.

Screenshots / Recordings
If applicable, add screenshots or short video recordings to help show the bug in more detail.

Additional context
Add any other context about the problem here.

@dmnk dmnk added the 🐛 Bug label Jan 23, 2025
@zachgoll
Copy link
Collaborator

@dmnk do you have a Synth API key setup for your instance? This is where we fetch ticker symbols from.

@zachgoll zachgoll added the ℹ️ Needs info Further information is requested label Jan 23, 2025
@dmnk
Copy link
Author

dmnk commented Jan 23, 2025

yes, is configured.
Displays "FREE PLAN" and 0 / 1000 API Calls used.

@zachgoll zachgoll removed the 🐛 Bug label Jan 24, 2025
@coldhearti
Copy link

Can confirm this. Can't add any holdings manually. Additionally, on CSV import symbols are not read/registered correctly. I also have some transactions for which the symbols probably are not available on Synth API. I suggest you allow adding holdings for which there are no tickers available. I fear this project may be trying to make some of the functionality way more complicated than it needs to be.

@dmnk
Copy link
Author

dmnk commented Jan 27, 2025

Console Error after providing the symbol.

Uncaught (in promise) TypeError: crypto.randomUUID is not a function
_enqueueCallback http://internal_maybe:3000/assets/hw_combobox/models/combobox/callbacks-4eb4299c.js:12
_filterAsync http://internal_maybe:3000/assets/hw_combobox/models/combobox/filtering-d34c6423.js:46
_debouncedFilterAsync http://internal_maybe:3000/assets/hw_combobox/models/combobox/filtering-d34c6423.js:38
callback http://internal_maybe:3000/assets/hw_combobox/helpers-8ec22533.js:44
setTimeout handler*debounce/< http://internal_maybe:3000/assets/hw_combobox/helpers-8ec22533.js:46
_filter http://internal_maybe:3000/assets/hw_combobox/models/combobox/filtering-d34c6423.js:29
filterAndSelect http://internal_maybe:3000/assets/hw_combobox/models/combobox/filtering-d34c6423.js:8
invokeWithEvent http://internal_maybe:3000/assets/stimulus.min-4b1e420e.js:4
handleEvent http://internal_maybe:3000/assets/stimulus.min-4b1e420e.js:4
handleEvent http://internal_maybe:3000/assets/stimulus.min-4b1e420e.js:4
connect http://internal_maybe:3000/assets/stimulus.min-4b1e420e.js:4
createEventListener http://internal_maybe:3000/assets/stimulus.min-4b1e420e.js:4
fetchEventListener http://internal_maybe:3000/assets/stimulus.min-4b1e420e.js:4
fetchEventListenerForBinding http://internal_maybe:3000/assets/stimulus.min-4b1e420e.js:4
bindingConnected http://internal_maybe:3000/assets/stimulus.min-4b1e420e.js:4
connectAction http://internal_maybe:3000/assets/stimulus.min-4b1e420e.js:4
elementMatchedValue http://internal_maybe:3000/assets/stimulus.min-4b1e420e.js:4
tokenMatched http://internal_maybe:3000/assets/stimulus.min-4b1e420e.js:4
tokenMatched http://internal_maybe:3000/assets/stimulus.min-4b1e420e.js:4
tokensMatched http://internal_maybe:3000/assets/stimulus.min-4b1e420e.js:4
tokensMatched http://internal_maybe:3000/assets/stimulus.min-4b1e420e.js:4
elementMatchedAttribute http://internal_maybe:3000/assets/stimulus.min-4b1e420e.js:4
elementMatched http://internal_maybe:3000/assets/stimulus.min-4b1e420e.js:4
addElement http://internal_maybe:3000/assets/stimulus.min-4b1e420e.js:4
refresh http://internal_maybe:3000/assets/stimulus.min-4b1e420e.js:4
start http://internal_maybe:3000/assets/stimulus.min-4b1e420e.js:4
start http://internal_maybe:3000/assets/stimulus.min-4b1e420e.js:4
start http://internal_maybe:3000/assets/stimulus.min-4b1e420e.js:4
start http://internal_maybe:3000/assets/stimulus.min-4b1e420e.js:4
start http://internal_maybe:3000/assets/stimulus.min-4b1e420e.js:4
connect http://internal_maybe:3000/assets/stimulus.min-4b1e420e.js:4
connectContextForScope http://internal_maybe:3000/assets/stimulus.min-4b1e420e.js:4
scopeConnected http://internal_maybe:3000/assets/stimulus.min-4b1e420e.js:4
elementMatchedValue http://internal_maybe:3000/assets/stimulus.min-4b1e420e.js:4
tokenMatched http://internal_maybe:3000/assets/stimulus.min-4b1e420e.js:4
tokenMatched http://internal_maybe:3000/assets/stimulus.min-4b1e420e.js:4
tokensMatched http://internal_maybe:3000/assets/stimulus.min-4b1e420e.js:4
tokensMatched http://internal_maybe:3000/assets/stimulus.min-4b1e420e.js:4
elementMatchedAttribute http://internal_maybe:3000/assets/stimulus.min-4b1e420e.js:4
elementMatched http://internal_maybe:3000/assets/stimulus.min-4b1e420e.js:4
addElement http://internal_maybe:3000/assets/stimulus.min-4b1e420e.js:4
processTree http://internal_maybe:3000/assets/stimulus.min-4b1e420e.js:4
processAddedNodes http://internal_maybe:3000/assets/stimulus.min-4b1e420e.js:4
processMutation http://internal_maybe:3000/assets/stimulus.min-4b1e420e.js:4
processMutations http://internal_maybe:3000/assets/stimulus.min-4b1e420e.js:4
mutationObserver http://internal_maybe:3000/assets/stimulus.min-4b1e420e.js:4
f http://internal_maybe:3000/assets/stimulus.min-4b1e420e.js:4
b http://internal_maybe:3000/assets/stimulus.min-4b1e420e.js:4
N http://internal_maybe:3000/assets/stimulus.min-4b1e420e.js:4
F http://internal_maybe:3000/assets/stimulus.min-4b1e420e.js:4
J http://internal_maybe:3000/assets/stimulus.min-4b1e420e.js:4
H http://internal_maybe:3000/assets/stimulus.min-4b1e420e.js:4
Q http://internal_maybe:3000/assets/stimulus.min-4b1e420e.js:4
start http://internal_maybe:3000/assets/stimulus.min-4b1e420e.js:4
http://internal_maybe:3000/assets/controllers/application-0efb88a1.js:3
8 callbacks-4eb4299c.js:12:31

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ℹ️ Needs info Further information is requested
Projects
None yet
Development

No branches or pull requests

3 participants