Penpot/penpot
0
Fork 0
mirror of https://github.com/penpot/penpot.git synced 2025-01-10 08:50:57 -05:00
Code Issues Activity

✨ Improve audit log for create files

Browse source
This commit is contained in:
Alejandro Alonso 2022-07-27 11:57:27 +02:00
parent 1eede8442d
commit bfe54fe5e1
5 changed files with 19 additions and 14 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

9
frontend/src/app/main/data/dashboard.cljs
View file

@ -713,18 +713,21 @@
[{:keys [project-id] :as params}] [{:keys [project-id] :as params}]
(us/assert ::us/uuid project-id) (us/assert ::us/uuid project-id)
(ptk/reify ::create-file (ptk/reify ::create-file
IDeref
(-deref [_] {:project-id project-id})
ptk/WatchEvent ptk/WatchEvent
(watch [_ _ _] (watch [it _ _]
(let [{:keys [on-success on-error] (let [{:keys [on-success on-error]
:or {on-success identity :or {on-success identity
on-error rx/throw}} (meta params) on-error rx/throw}} (meta params)
name (name (gensym (str (tr "dashboard.new-file-prefix") " "))) name (name (gensym (str (tr "dashboard.new-file-prefix") " ")))
params (assoc params :name name)] params (assoc params :name name)]
(->> (rp/mutation! :create-file params) (->> (rp/mutation! :create-file params)
(rx/tap on-success) (rx/tap on-success)
(rx/map file-created) (rx/map #(with-meta (file-created %) (meta it)))
(rx/catch on-error)))))) (rx/catch on-error))))))
;; --- EVENT: duplicate-file ;; --- EVENT: duplicate-file

1
frontend/src/app/main/data/events.cljs
View file

@ -84,6 +84,7 @@
(derive :app.main.data.comments/update-comment-thread-status ::generic-action) (derive :app.main.data.comments/update-comment-thread-status ::generic-action)
(derive :app.main.data.dashboard/delete-team-member ::generic-action) (derive :app.main.data.dashboard/delete-team-member ::generic-action)
(derive :app.main.data.dashboard/duplicate-project ::generic-action) (derive :app.main.data.dashboard/duplicate-project ::generic-action)
(derive :app.main.data.dashboard/create-file ::generic-action)
(derive :app.main.data.dashboard/file-created ::generic-action) (derive :app.main.data.dashboard/file-created ::generic-action)
(derive :app.main.data.dashboard/invite-team-members ::generic-action) (derive :app.main.data.dashboard/invite-team-members ::generic-action)
(derive :app.main.data.dashboard/leave-team ::generic-action) (derive :app.main.data.dashboard/leave-team ::generic-action)

7
frontend/src/app/main/ui/dashboard/files.cljs
View file

@ -75,7 +75,7 @@
:on-import on-import}] :on-import on-import}]
[:div.dashboard-header-actions [:div.dashboard-header-actions
[:a.btn-secondary.btn-small {:on-click (partial on-create-clicked project) :data-test "new-file"} [:a.btn-secondary.btn-small {:on-click (partial on-create-clicked project "dashboard:header") :data-test "new-file"}
(tr "dashboard.new-file")] (tr "dashboard.new-file")]
(when-not (:is-default project) (when-not (:is-default project)
@ -100,9 +100,10 @@
on-create-clicked on-create-clicked
(mf/use-callback (mf/use-callback
(fn [project event] (fn [project origin event]
(dom/prevent-default event) (dom/prevent-default event)
(st/emit! (dd/create-file {:project-id (:id project)}))))] (st/emit! (with-meta (dd/create-file {:project-id (:id project)})
{::ev/origin origin}))))]
(mf/use-effect (mf/use-effect
(mf/deps project) (mf/deps project)

2
frontend/src/app/main/ui/dashboard/placeholder.cljs
View file

@ -19,7 +19,7 @@
:else :else
[:div.grid-empty-placeholder [:div.grid-empty-placeholder
[:button.create-new {:on-click (partial on-create-clicked project)} [:button.create-new {:on-click (partial on-create-clicked project "dashboard:empty-folder-placeholder")}
(tr "dashboard.new-file")]])) (tr "dashboard.new-file")]]))
(mf/defc loading-placeholder (mf/defc loading-placeholder

10
frontend/src/app/main/ui/dashboard/projects.cljs
View file

@ -93,10 +93,11 @@
create-file create-file
(mf/use-callback (mf/use-callback
(mf/deps project) (mf/deps project)
(fn [] (fn [origin]
(let [mdata {:on-success on-file-created} (let [mdata {:on-success on-file-created}
params {:project-id (:id project)}] params {:project-id (:id project)}]
(st/emit! (dd/create-file (with-meta params mdata)))))) (st/emit! (with-meta (dd/create-file (with-meta params mdata))
{::ev/origin origin})))))
on-import on-import
(mf/use-callback (mf/use-callback
@ -140,7 +141,7 @@
i/pin)]) i/pin)])
[:a.btn-secondary.btn-small.tooltip.tooltip-bottom [:a.btn-secondary.btn-small.tooltip.tooltip-bottom
{:on-click create-file :alt (tr "dashboard.new-file") :data-test "project-new-file"} {:on-click (partial create-file "dashboard:folder") :alt (tr "dashboard.new-file") :data-test "project-new-file"}
i/close] i/close]
[:a.btn-secondary.btn-small.tooltip.tooltip-bottom [:a.btn-secondary.btn-small.tooltip.tooltip-bottom
@ -152,8 +153,7 @@
:team team :team team
:on-load-more on-nav :on-load-more on-nav
:files files :files files
:on-create-clicked create-file}]])) :on-create-clicked (partial create-file "dashboard:empty-folder-placeholder")}]]))
(def recent-files-ref (def recent-files-ref
(l/derived :dashboard-recent-files st/state)) (l/derived :dashboard-recent-files st/state))