problem: [/currency/x] not possible to propose adding exchanges to a currency

[gsovereignty]

Problem: it's not possible for users to add exchanges to coins on the currency detail page.

possible solution: Allow users to add an exchange to a currency from the existing list of exchanges or add a new exchange. Ideally this would be a typahead list with "add" if what the user types isn't found (e.g. how Github works with adding branches).

Any solution should also denormalize the data into the currencies collection so that currencies on the home route can be filtered by "is currently listed on an exchange".

[janat08]

All the logic for this is done, except for the add template and some other issues. Notes in last comments of #783.

[samuelralak]

whats the progress on this one? is anyone actively working on this one?

[gsovereignty]

No, no one has claimed it

[janat08]

I think I saw jquery typeahead that was reactive for meteor, that might solve issues.

[gsovereignty]

Can you find it again?

[samuelralak]

ok cool. can't claim now though but will take a look if its not claimed before finishing my current issues

No, no one has claimed it

[janat08]

https://atmospherejs.com/sergeyt/typeahead
I'll take it if we're converting.

[gsovereignty]

Just remember there are no secret updates and no discussions outside of the issue tracker, so what you see here is what you get. If it's not claimed, no one is working on it. And if there are no attached pull requests and no comments, there are no updates ;)

[janat08]

It said it supported blaze templates for autocomplete suggestions, but it's likely that the typeahead is outdated and so might not have the empty template option.

[gsovereignty]

@janat08 That looks like the simplest solution :)

[gsovereignty]

If there are any problems with that package and you want to modify it I'll fork it here and C4 it, it looks like there's a good pull request just waiting to be merged but the maintainer is being picky about silly things.

[janat08]

Still can't have reactive value since I can't bind an event to typeahead input: sergeyt/meteor-typeahead#161

[janat08]

@blockrazorbot claim

[gsovereignty]

@janat08 can you remind me, what's the problem with the existing typeahead?

[janat08]

I couldn't pass in a event or reflect current value of typeahead in empty template.

[gsovereignty]

in empty template.

Can you explain more here, a template referring to an empty data set?

[janat08]

empty result set, prompting to make a new result.

[gsovereignty]

OK. I don't know how far along you are with this, but a simpler solution will be to just add a "add new" button next to the typeahead dropdown (similar to /add-hashpower).

[janat08]

The package is in place, it's the fact that I can't select typeahead element or update the value aside. The typeahead('val') will work, it's more about other options I like custom events and updating the value for reactivity.

[gsovereignty]

OK. Stay focussed on the problem. This solution is probably too complicated and not worth it.

If you have to make something work in an hour and close this issue what would you do?

[janat08]

Ideally this would be a typahead list with "add" if what the user types isn't found (e.g. how Github works with adding branches).
Do you want a separate button? If element selection were to work it wouldn't take over an hour to finish.

[gsovereignty]

Oh ok, well element selection would be better :)

[janat08]

I'm just waiting for someone to respond to sergeyt/meteor-typeahead#161 though.

[gsovereignty]

It's safe to say no one is going to respond. So just do whatever the simplest thing is to close this off.

[gsovereignty]

Closed and moved to #1236