Added href to field ID linking to ONIX FIX dictionary

drewnoakes / fix-decoder

Unravels FIX messages into human readable tables

80 stars 34 forks source link

Added href to field ID linking to ONIX FIX dictionary #12

Closed chrismasters closed 10 years ago

chrismasters commented 10 years ago

Hey, fairly simple, not sure if you're happy to take it, but I find myself looking up tags quite often and linking them seemed like a minimal thing.

drewnoakes commented 10 years ago

Looks great! I rebased rather than merged your commit so GitHub can't tell that the PR has been accepted. My bad, sorry. Anyway your commit's on master here: 54cea8d71ccf7e5afc0a932599067c1b8f33467a

Ideally it'd be great to bring the descriptions and whatever else information into the data.js file so that everything's in one place. Linking to ONIX is an elegant solution.

drewnoakes commented 10 years ago

BTW the email address you committed under isn't linked to your GitHub account, so the commit isn't attributed to you properly. If you update your account settings it should link up properly.