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

fix: go formatting stuff werent running through entire directory #23

Merged
merged 1 commit into from
Feb 1, 2024

Conversation

MattCMcCoy
Copy link
Contributor

Description

Go stuff in task format werent running through entire directory, bug was found after I got spammed with emails about formatting stuff, in the current pr for the task group

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. If they are unit
tests, provide the file name the tests are in. If they are not unit tests,
describe how you tested the change.

Switched to their branch and spammed different changes until one worked tbh

Checklist

  • I have performed a self-review of my code
  • I have reached out to another developer to review my code
  • I have commented my code, particularly in hard-to-understand areas
  • New and existing unit tests pass locally with my changes

@MattCMcCoy MattCMcCoy merged commit 381af34 into main Feb 1, 2024
3 checks passed
@MattCMcCoy MattCMcCoy deleted the bugfix/fix-formatting-task branch February 1, 2024 22:48
oliviaseds added a commit that referenced this pull request Feb 17, 2024
* Added mock data

* feat: get tasks by gid/uid/status/type/date

* style: formatting

* fix: Remove commitizen branch

* test: fix task tests

* style: quick reformat

* feat: assign user(s) to a task CRUD and test

* style: quick style

* refactor: abstract queries, enable multiple

* refactor: abstract filter query/arg building logic

* crud routes for create new task, delete task, and update task info

* Feat: generic card and popup modal (#20)

* Intial card/button

* Basic card and modal

Basic card and modal skeleton

* fixed card and popup to have the requirments

fixed card and popup to have the requirments

* wip for passing medication data into Card component

* trying to get stuff workin

* trying to get comps workin

* stuff

* Final Working Card + Popup

---------

Co-authored-by: AdharshKan42 <[email protected]>

* fix: go formatting stuff werent running through entire directory (#23)

* feat: add a user context provider to the app (#24)

* feat: userContext stuff idk man

* refactor: general all around fixes (removing unused imports, fixing navigation errors)

* refactor: all over the place with this commit, but added swagger docs for file upload

* refactor: make context more general

* refactor: utilize card and popup in the medlist FE

* backend hot reload

* feat: added hot reloading for backend code

* refactor: abstract queries, enable multiple

* refactor: routes are updated

* feat: new route backend

* feat: add a new label crud and test

* feat: delete label crud and test

* refactor: FE  remove extra card file

* test: fix typo in db, tests passing locally

* refactor: modify task filter to bind to query

* feat: update task name and color route and test

* docs: fix swaggo comment style for task assignment and deletion

* docs: same swagger changes for label routes

* refactor: implement inclusive data range filter

* refactor: keep consistency between route groups

* docs: fix a swagger doc return type

---------

Co-authored-by: bedrockdude10 <[email protected]>
Co-authored-by: haleymartin-6 <[email protected]>
Co-authored-by: CaitlinFlynn <[email protected]>
Co-authored-by: AdharshKan42 <[email protected]>
Co-authored-by: Matt McCoy <[email protected]>
Co-authored-by: Matt McCoy <[email protected]>
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.

1 participant