Skip to content

[web] Incorporate Vuex to UI#282

Open
AtoBrightSide wants to merge 15 commits intomainfrom
abel.blogUI
Open

[web] Incorporate Vuex to UI#282
AtoBrightSide wants to merge 15 commits intomainfrom
abel.blogUI

Conversation

@AtoBrightSide
Copy link

No description provided.

@vercel
Copy link

vercel bot commented Aug 4, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
practice ✅ Ready (Inspect) Visit Preview Aug 5, 2022 at 8:21AM (UTC)
practice-project-vue ✅ Ready (Inspect) Visit Preview Aug 5, 2022 at 8:21AM (UTC)
practice-vue ✅ Ready (Inspect) Visit Preview Aug 5, 2022 at 8:21AM (UTC)

samiabat and others added 2 commits August 4, 2022 15:49
* main branch

* create folder to my project

* add crud

* fix post issue

* finish create operation

* add post

* update completed

* Update SamuelBlogPost.vue

string literal rather than concat

* Update SamuelAddBlog.vue

remove unnecessary code.

* Update SamuelAddBlog.vue

change by object and remove unnecessary code

* Update SamuelBlogPost.vue

* Update SamuelBlogPost.vue

* Update samuel.js

* restore

* fix issue

* remove unnessasry code

* add changes
* [web] implement blog list

* [web] minor refractions made

* [web] implemented authentication

* [web] lintfix

* [web] implemented authentication

* [web] refractions made on auth

* [web] fixed comments on PR

* Yared.demo project UI (#270)

* create the UI

* [web] implement vuex state management

* [web] implement CRUD functionalities

* [web] add authentication and fix previous comments

* [WEB]: folder restructure to work with team

* [web]: sample structure for the demo

* [web]: folder rename to PascalCase

* [web]: remove feature folder from store

* [web]: fix a bug inside index page

* [web]: fix deployment bug

* Yohans.web.demo (#256)

* [web][yohans] added some basic ui and simple database mocking array

* [web][yohans] added some state for the blogs

* [WEB]: folder restructure 

* [web]: sample structure for the demo

* [web]: remove feature folder from store

* [web]: file rename issue with git solved

* [WEB]: fix deploy error

* [web] implemented authentication

* [web] refractions made on auth

* [web] implement blog list

* [web] implemented authentication

* [web] lintfix

* [web] fixed comments on PR

* [web] comments fixed and lintfix

Co-authored-by: Yared <90713646+Yared04@users.noreply.github.com>
Co-authored-by: Mintesnot Bezabhi <61611671+Mintaa911@users.noreply.github.com>
Co-authored-by: Yohans Hailu <95067604+YohansHailu@users.noreply.github.com>
Comment on lines 5 to 8
:title="blog.title"
:fullName="blog.authorUserId.fullName"
:tag="blog.tag"
:content="blog.content"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Instead of passing individual properties, it is better to pass the object as props

@@ -0,0 +1,26 @@
<template>
<v-card class="mx-auto deep-purple" min-height="250" max-width="344">
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The texts on the cards are not readable. Go for something lighter.

@@ -0,0 +1,26 @@
<template>
<v-card class="mx-auto deep-purple" min-height="250" max-width="344">
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The texts on the cards are not readable. Go for something lighter.

@@ -0,0 +1,33 @@
<template>
<button @click="$emit('toggle-task')">{{ text }}</button>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would recommend using a vuetify button. and using classes to make it look how you want it to. check more about it here but if you needed this exact component, It's ok

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants