Skip to content
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

UI update and preparing automatic builds. #11

Merged
merged 9 commits into from
Mar 25, 2024
Merged

UI update and preparing automatic builds. #11

merged 9 commits into from
Mar 25, 2024

Conversation

Makosai
Copy link
Owner

@Makosai Makosai commented Mar 25, 2024

This pull request handles the following:

The README mostly talks about a roadmap for the project showing potential features. The UI, if it's implemented on the web, will also fulfill the mobile UI. So after all of the pages are built, I'll make them responsive for mobile and tablet screens.

This pull request also opens up the pathway to fixing the automatic build process for #10.

Copy link

vercel bot commented Mar 25, 2024

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

Name Status Preview Updated (UTC)
visp ✅ Ready (Inspect) Visit Preview Mar 25, 2024 1:42am

@Makosai Makosai self-assigned this Mar 25, 2024
@Makosai Makosai added type: feature Brand new functionality, features, pages, workflows, endpoints, etc. priority: now effort: 2 semantic-release work: obvious The situation is obvious, best practices used. labels Mar 25, 2024
@Makosai Makosai modified the milestone: Produce an MVP Mar 25, 2024
src/modules/video/mod.rs Dismissed Show dismissed Hide dismissed
@@ -1,7 +1,37 @@
use dioxus::html::HasFileData;

Check failure

Code scanning / clippy

failed to resolve: use of undeclared crate or module dioxus Error

failed to resolve: use of undeclared crate or module dioxus
use dioxus::prelude::*;
use crate::video::manager::get_video_file;

Check failure

Code scanning / clippy

unresolved import crate::video::manager::get_video_file Error

unresolved import crate::video::manager::get\_video\_file
@@ -3,32 +3,120 @@

use crate::routes::Route;

// Profile component
#[component]

Check failure

Code scanning / clippy

cannot find attribute component in this scope Error

cannot find attribute component in this scope
// Profile component
#[component]
pub(crate) fn Profile() -> Element {
rsx! {

Check failure

Code scanning / clippy

cannot find macro rsx in this scope Error

cannot find macro rsx in this scope
@@ -0,0 +1,16 @@
#[cfg_attr(target_family = "windows", path = "desktop_video_manager.rs")]
#[cfg_attr(target_family = "wasm", path = "web_video_manager.rs")]
pub mod manager;

Check warning

Code scanning / clippy

file is loaded as a module multiple times: src/main.rs Warning

file is loaded as a module multiple times: src/main.rs
pub (in crate::routes) fn Home() -> Element {
let mut count = use_signal(|| 0);
pub(in crate::routes) fn Home() -> Element {
let username = use_signal(|| "Kristopher");

Check failure

Code scanning / clippy

cannot find function use_signal in this scope Error

cannot find function use\_signal in this scope

#[component]
pub(in crate::routes) fn Video() -> Element {
let mut enable_directory_upload = use_signal(|| false);

Check failure

Code scanning / clippy

cannot find function use_signal in this scope Error

cannot find function use\_signal in this scope

#[component]
pub(in crate::routes) fn Video() -> Element {
let mut enable_directory_upload = use_signal(|| false);
let mut video_files_uploaded = use_signal(|| Vec::new() as Vec<VideoFile>);

Check failure

Code scanning / clippy

cannot find function use_signal in this scope Error

cannot find function use\_signal in this scope
#[component]
pub(crate) fn NavBar() -> Element {
// keep track of which button was pressed last
let mut selected = use_signal(|| "Home");

Check failure

Code scanning / clippy

cannot find function use_signal in this scope Error

cannot find function use\_signal in this scope
@Makosai Makosai linked an issue Mar 25, 2024 that may be closed by this pull request
4 tasks
@Makosai Makosai merged commit ed54b62 into main Mar 25, 2024
3 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
effort: 2 priority: now semantic-release type: feature Brand new functionality, features, pages, workflows, endpoints, etc. work: obvious The situation is obvious, best practices used.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Create a README
1 participant