Week 3 #3

Merged

lubba64 merged 13 commits from week2-additional into main 2026-06-24 16:36:44 +00:00

Conversation 12 Commits 13 Files changed 39 +1511 -1026

lubba64 commented 2026-06-20 01:19:29 +00:00

Owner

Copy link

Finishing some extra features before week 3 content so there's more stuff to test in the app.

Finishing some extra features before week 3 content so there's more stuff to test in the app.

lubba64 self-assigned this 2026-06-20 01:19:30 +00:00

lubba64 commented 2026-06-20 01:26:42 +00:00

Author

Owner

Copy link

working on app locking up. due to the way egui works, we can't really put the long sync task off to another thread (as far as I know). the main reason the thread locks is because the server takes quite a while to give its response in some cases. This makes sense because one of the routes does OCR, e.t.c. I'm going to figure out how to return early and send the task to a different async task on the server.

working on app locking up. due to the way egui works, we can't really put the long sync task off to another thread (as far as I know). the main reason the thread locks is because the server takes quite a while to give its response in some cases. This makes sense because one of the routes does OCR, e.t.c. I'm going to figure out how to return early and send the task to a different async task on the server.

lubba64 commented 2026-06-20 01:35:26 +00:00

Author

Owner

Copy link

this is going to create some problems when dealing with feedback for loading/errors but my hand has been forced, plus we can figure it out.

this is going to create some problems when dealing with feedback for loading/errors but my hand has been forced, plus we can figure it out.

lubba64 added 1 commit 2026-06-20 01:44:16 +00:00

fix: Fix the app freezing due to long async process times 1808c9f5d2

lubba64 commented 2026-06-20 01:44:37 +00:00

Author

Owner

Copy link

that appears to have solved the issue...

that appears to have solved the issue...

lubba64 commented 2026-06-20 01:45:19 +00:00

Author

Owner

Copy link

going to focus on feedback for status plus errors now.

going to focus on feedback for status plus errors now.

lubba64 commented 2026-06-20 02:26:03 +00:00

Author

Owner

Copy link

I'm feeling tired, I'll pick up again in the morning.

I'm feeling tired, I'll pick up again in the morning.

lubba64 commented 2026-06-20 17:35:30 +00:00

Author

Owner

Copy link

back at it, I have a bad habit of getting distracted.

back at it, I have a bad habit of getting distracted.

lubba64 commented 2026-06-20 17:36:01 +00:00

Author

Owner

Copy link

lets do some features and take care of status later

lets do some features and take care of status later

lubba64 added 1 commit 2026-06-20 18:52:54 +00:00

fix: Fix duplicate start dates replace with actual end date f0ff4fa5ff

lubba64 added 1 commit 2026-06-20 19:39:43 +00:00

fix: Fix issue with date format, change default dates for manual picker to be Local::now() b2a9ff9850

lubba64 added 1 commit 2026-06-20 20:37:55 +00:00

refactor: Make crate for common err logic, refactor. add schema for labels 377b484d6a

lubba64 added 1 commit 2026-06-20 23:24:22 +00:00

feat: Label routes 0daf4fbaca

lubba64 added 1 commit 2026-06-20 23:47:38 +00:00

feat: routes/tables for task labels 40f1c5bb3d

lubba64 added 1 commit 2026-06-20 23:54:10 +00:00

refactor: Move some scripts around, add missing scripts back f55374a1f7

lubba64 commented 2026-06-21 00:19:04 +00:00

Author

Owner

Copy link

I think composing objects on the server is probably better, but eh, I can always refactor later. I didn't make all those routes for nothing (we need them anyways for editing etc)

I think composing objects on the server is *probably* better, but eh, I can always refactor later. I didn't make all those routes for nothing (we need them anyways for editing etc)

lubba64 commented 2026-06-21 00:23:01 +00:00

Author

Owner

Copy link

composing objects on the server is going to be easier I suppose since the client side logic for resolving promises is terrible.

composing objects on the server is going to be easier I suppose since the client side logic for resolving promises is terrible.

lubba64 changed title from Week 2 Cleanup to Week 2 Additional Features 2026-06-21 01:10:57 +00:00

lubba64 added 1 commit 2026-06-21 01:28:53 +00:00

chore: Save progress on labels server infrastructure c600ed4281

lubba64 added 1 commit 2026-06-21 02:08:05 +00:00

fix: Fix refreshing being borked 8823c8823c

lubba64 changed title from Week 2 Additional Features to Week 3 2026-06-21 02:24:42 +00:00

lubba64 added 1 commit 2026-06-22 04:09:03 +00:00

feat: basic label adding f5b014b125

lubba64 added 1 commit 2026-06-22 05:10:08 +00:00

chore: Save while some bs happens 33bd51e1ac

lubba64 commented 2026-06-22 05:44:37 +00:00

Author

Owner

Copy link

another 4 hours and the feature is mostly complete.... however, the window is now failing to create between the second to last commit and the current one, going to get some sleep and tackle this in the morning. I will likely move forward with a version for the think aloud tests that does not have the labels so I can make sure my time is used effectively.

another 4 hours and the feature is mostly complete.... however, the window is now failing to create between the second to last commit and the current one, going to get some sleep and tackle this in the morning. I will likely move forward with a version for the think aloud tests that does not have the labels so I can make sure my time is used effectively.

lubba64 added 1 commit 2026-06-22 23:56:46 +00:00

chore: Save bs 895fe15705

lubba64 commented 2026-06-24 15:09:39 +00:00

Author

Owner

Copy link

I'm going to fix the bug, merge it and make a new pr

I'm going to fix the bug, merge it and make a new pr

lubba64 commented 2026-06-24 16:28:05 +00:00

Author

Owner

Copy link

apparently egui hated my attempt to be DRY with this blob, couldn't tell you why that is.

        let cloned_refresh = self.last_refresh.clone();
        let mut fetch = || {
            self.get_tasks = Some(get_json("task/getallfull".to_string()));
            self.get_labels = Some(get_json::<Label>("/label/getall".to_string()));
            match cloned_refresh.lock() {
                Ok(mut instant) => *instant = Some(Instant::now()),
                Err(_err) => {}
            }
        };

        match self.last_refresh.lock() {
            Ok(instant) => match *instant {
                Some(instant) => {
                    if instant.elapsed() > Duration::from_secs(1) {
                        fetch()
                    }
                }
                None => fetch(),
            },
            Err(_err) => {}

apparently `egui` hated my attempt to be DRY with this blob, couldn't tell you why that is. ```rust let cloned_refresh = self.last_refresh.clone(); let mut fetch = || { self.get_tasks = Some(get_json("task/getallfull".to_string())); self.get_labels = Some(get_json::<Label>("/label/getall".to_string())); match cloned_refresh.lock() { Ok(mut instant) => *instant = Some(Instant::now()), Err(_err) => {} } }; match self.last_refresh.lock() { Ok(instant) => match *instant { Some(instant) => { if instant.elapsed() > Duration::from_secs(1) { fetch() } } None => fetch(), }, Err(_err) => {} ```

lubba64 added 1 commit 2026-06-24 16:36:28 +00:00

fix: Fix window not creating due to some unknown bs a64a2a7935

lubba64 merged commit 791aeb944b into main 2026-06-24 16:36:44 +00:00

lubba64 referenced this pull request from a commit 2026-06-24 16:36:45 +00:00

Merge pull request 'Week 3' (#3) from week2-additional into main

Sign in to join this conversation.

Reviewers

No reviewers

Labels

Clear labels   

bug

Something isn't working

  

documentation

Improvements or additions to documentation

  

duplicate

This issue or pull request already exists

  

enhancement

New feature or request

  

good first issue

Good for newcomers

  

help wanted

Extra attention is needed

  

invalid

This doesn't seem right

  

question

Further information is requested

  

wontfix

This will not be worked on

No labels

bug

documentation

duplicate

enhancement

good first issue

help wanted

invalid

question

wontfix

Milestone

Clear milestone

No items

No milestone

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

lubba64

1 participant

Notifications

Subscribe

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

lubba64/Taskify!3

No description provided.