learn-re-frame-course-files icon indicating copy to clipboard operation
learn-re-frame-course-files copied to clipboard

complete version not working properly

Open michael-mhrdd opened this issue 4 years ago • 1 comments

When running the complete version locally, it differs from app.learnreframe.com: After agreeing to become a chef and adding a recipe, then opening it,

  • the green button at the at the top right says "Order for NaN €" instead of "Publish".
  • the Steps + and Ingredients + buttons have no effect (I can't add ingredients and steps).
  • When navigating back to recipes, then new recipe has disappeared.

npm install produces these warnings: npm WARN deprecated [email protected]: Support has ended for 9.x series. Upgrade to @latest npm WARN deprecated [email protected]: [email protected] is no longer supported. Please upgrade to @latest f or important bug and security fixes.

at the end of npm run dev, i get: Warning: Nashorn engine is planned to be removed from a future JDK release

Also, the re-frame-10x tab is strangely formatted:

image

michael-mhrdd avatar Mar 27 '21 23:03 michael-mhrdd

As for the re-frame-10x tab what I did to fix that whole UI was

was changing re-frame version in shadow-cljs.edn

to

[day8.re-frame/re-frame-10x "1.2.1"]

and the reagent version to

[reagent "1.1.0"]

this also required me to update the the core.cljs file to the following

(ns app.core
  (:require [reagent.core :as r]
            ;; addition here
            [reagent.dom  :as rd]))

(defn app
  []
  [:div "Cheffy"])

(defn ^:dev/after-load start
  []
  ;; change to rd
  (rd/render [app]
    (.getElementById js/document "app")))

(defn ^:export init
  []
  (start))

remember to stop and start server, either npm run dev or yarn dev

I am also running node 12 so that may help with the node warnings.

As for the functionality of the rest of the features I am not sure, but will message when I get to that part, I have started over because my REPL was way to broken and couldn't fix it.

Anyways hope this is of some help

anderskitson avatar Mar 27 '22 19:03 anderskitson