feature/show-project-detail #74

aniram · 2026-03-05T21:06:30Z

aniram commented

2026-03-05 21:06:30 +00:00

Some major refactoring happened:

Added the project's detail page
Added scrolling inside the project's history
Models no longer have properties with the 'CID' prefix whose purpose was to indicate that this a property that has been artificially glued to the model since although they belong together contextually they do not do so through the API. E.g.: A Project has an Author. So Project would have a CIDAuthor property. This allowed for the mistake of creating bloated templates that had access to everything although they only needed a fraction of the data passed.
Made the templates "dummer" so they only know what they need to and can be snapped to another template like lego pieces.
Added a template function called 'dict' that helps pass multiple information to a template. I'm not happy with this solution. I would rather have a smarter template engine and also the ability to use slots. Maybe in the future this app should be rewritten with Django or a similar framework with an established template engine.
Removed the use of mutexes in order to breakdown all of the functions that make requests to the API to make them reusable. Now everything is being passed through a channel. There is one little hack which is that since I'm fetching all authors in one goroutine and all projects in another goroutine, it is hard to associate the project to its main author so in order to do that when I fetch the author I also pass the index of the project, and then later I associate them through this index. It's dirty but it works for now.

Some major refactoring happened: - Added the project's detail page - Added scrolling inside the project's history - Models no longer have properties with the 'CID' prefix whose purpose was to indicate that this a property that has been artificially glued to the model since although they belong together contextually they do not do so through the API. E.g.: A Project has an Author. So Project would have a CIDAuthor property. This allowed for the mistake of creating bloated templates that had access to everything although they only needed a fraction of the data passed. - Made the templates "dummer" so they only know what they need to and can be snapped to another template like lego pieces. - Added a template function called 'dict' that helps pass multiple information to a template. I'm not happy with this solution. I would rather have a smarter template engine and also the ability to use slots. Maybe in the future this app should be rewritten with Django or a similar framework with an established template engine. - Removed the use of mutexes in order to breakdown all of the functions that make requests to the API to make them reusable. Now everything is being passed through a channel. There is one little hack which is that since I'm fetching all authors in one goroutine and all projects in another goroutine, it is hard to associate the project to its main author so in order to do that when I fetch the author I also pass the index of the project, and then later I associate them through this index. It's dirty but it works for now.

aniram added 38 commits

2026-03-05 21:06:30 +00:00

feat: add proof-of-concept for the use of the VirtualKeyboard API e4022ef361

refactor: minor clean-up bc9629c9ce

feat: add ugly and unfinished skeleton of details page e4bdd7312a

feat: add avatar and head 725134100e

refactor: minor fad8670fe6

fix: do not show image if project author is a commission (#62 ) 0465181c12

refactor: rename project state properties 477ad87f32

feat: fetch and render project's history bab2c85fe1

feat: self-host HTMX library 50761fdb48

feat: click on project description opens detail page 2229aaad97

feat: show loading spinner when fetching project history with HTMX + show popover instead of dialog for actions menu 94dce4e195

feat: make project detail header sticky on mobile 8a8e7fe779

feat: add share button ee9d7b7f76

refactor: minor adjustments on font-sizes 3276dfc650

feat: scroll to top on when clicking on the project's name 49a40819d7

feat: format time to a more usual format for Brazilians and translate to Portuguese 3f505ec74f

refactor: minor 026a8def6d

feat: add polyfill for Invoker API 600efd7b86

fix: Safari only renders SVGs with a defined size 83e4d4599c

feat: should error message when HTMX request fails 91d17c2476

refactor: decouple templates into smaller ones d3d245520e

refactor: remove slash from HTML elements which are void 349700ac0c

refactor(css): add shadow to card 3f12d54e78

refactor(css): minor alignment 7e1333b9bf

refactor(css): increase vertical padding 5c46da8504

feat(wip): fetch and render project detail page 582b1ca932

refactor: minor 5499277427

refactor: decouple models from each other and simplify fetching of projects 2b14a4ac24

refactor: open project PDF on same tab a29fbd2fb6

refactor: remove typo 5d330d90d0

chore: gitignore idea folder 4584d2cf5f

refactor: minor 96b6bbd528

feat: add license to new template files d8718e00e3

refactor: minor d1b2175efc

fix: make navigation header of project detail fixed again b6b03f59f0

chore: add hot-reload to air 390466e1ea

refactor: minor d7e44a91d6

fix(css): align component labels and hide close button from project page d8702edffd

aniram added 1 commit

2026-03-05 21:11:45 +00:00

feat(css): scroll inside project history 5ec153d8b1

aniram added 1 commit

2026-03-05 21:26:51 +00:00

feat: add wait group to render project request efee6df432

aniram added 1 commit

2026-03-05 21:29:42 +00:00

feat(css): add the same padding to project page as the others have(detail, index) 7b03a44cd6

aniram force-pushed feature/show-project-detail from 7b03a44cd6 to 7934219918

2026-03-05 21:45:35 +00:00

Compare

aniram added 2 commits

2026-03-06 19:43:29 +00:00

refactor: minor renaming 796e403bd8

refactor: remove unnecessary closure of channels 3e0f8a82cd

aniram reviewed

2026-03-06 20:09:21 +00:00

internal/model/project.go Outdated

					
				@ -63,0 +73,4 @@

						return nil

					}

					// this layout is not optional, it is the closes to how the API returns date

typo

aniram reviewed

2026-03-06 20:12:56 +00:00

aniram left a comment

pretty good

aniram added 1 commit

2026-03-06 20:13:57 +00:00

refactor: remove typo 2ac93b7bee

aniram merged commit 808386de29 into main

2026-03-06 20:14:08 +00:00

aniram referenced this pull request from a commit

2026-03-06 20:14:10 +00:00

Merge pull request 'feature/show-project-detail' (#74) from feature/show-project-detail into main

Sign in to join this conversation.

No reviewers

No milestone

No project

No assignees

1 participant

Notifications

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

overdrives/cidadon!74

Rows
Columns