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
[Fixes #5496] Added SuperRare NFT support #5686
Conversation
@@ -19,7 +19,7 @@ | |||
(fn [{:keys [db]} [_ address {:keys [symbol amount] :as collectible}]] | |||
(let [items-number (money/to-number amount) | |||
loaded-items-number (count (get-in db [:collectibles symbol]))] | |||
(merge (when (not= items-number loaded-items-number) | |||
(merge (when true ;(not= items-number loaded-items-number) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
?
(fn [{db :db} [_ symbol collectibles]] | ||
{:db (update-in db [:collectibles symbol] merge collectibles)})) | ||
|
||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
extra line
src/status_im/utils/http.cljs
Outdated
@@ -15,7 +15,8 @@ | |||
(post url data on-success nil)) | |||
([url data on-success on-error] | |||
(post url data on-success on-error nil)) | |||
([url data on-success on-error {:keys [timeout-ms headers]}] | |||
([url data on-success on-error {:keys [valid-response? timeout-ms headers]}] | |||
(println "HEADERS" headers) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
?
59448ea
to
3892fb9
Compare
@flexsurfer Please take another look! |
@@ -33,6 +33,10 @@ | |||
address | |||
(str hex-prefix address)))) | |||
|
|||
(defn naked-address [s] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
naked-hex
maybe more common?
[react/image {:style (merge {:resize-mode :contain :width 100 :height 100} styles/details-image) | ||
:source {:uri imageUri | ||
:k 1.4}}] | ||
[react/view {:flex 1 :justify-content :center} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
move to styles? ¯_(ツ)_/¯
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
aren't we wasting energy creating dedicated style namespace for every views namespace, having to come up with names for styles, and in the end never reuse again because it is so specific ?
personally I would very much appreciate if we were doing the styling in-line, but ofc it is not always that easy and sometimes you have the defstyles macro for platform specific stuff so I don't have a strong opinion about it. But I sometime find it wasteful to follow the pattern that we have right now
A ropsten collectible 😮 👍 |
Builds failed with
https://ci.status.im/job/status-react/job/pull%20requests/job/PR-5686/ I'm restarting manually |
@jeluard the build is failing on formatting. Could you check?
|
@lukaszfryc Should be good now |
(and ok? (valid-response? response)) | ||
ok?)] | ||
[response-body ok?'])))))) | ||
(.then (fn [[response ok?]] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
what is this for ?
Signed-off-by: Julien Eluard <julien.eluard@gmail.com>
fixes #5496
Summary:
Added support for SuperRare NFT
status: ready