Code

Ron azar Resources and Services#22

Code

Merged

RonAzar merged 104 commits intomasterRonAzar/hibob-academy:masterfrom RonAzar/Resource-And-Service-OmerRonAzar/hibob-academy:RonAzar/Resource-And-Service-OmerCopy head branch name to clipboardSep 18, 2024

+429-36

Commits

Commits on Sep 8, 2024

• Not Finshed yet

  RonAzar

  committed
  a34c508

• Class Exercise-SQL

  RonAzar

  committed
  23f3ba6

Commits on Sep 10, 2024

• May's Review-the name of the file should be the date it was created

  RonAzar

  committed
  6224b5a

• May's Review-index should include the table name

  RonAzar

  committed
  5ab3350

• May's Review-where id=2

  RonAzar

  committed
  abe6e4d

• May's Review- DEFAULT date value

  RonAzar

  committed
  51edf29

• May's Review- rename to id, the owner part is redundant in the context of owner

  RonAzar

  committed
  c18f7f4

• May's Review- should be not null

  RonAzar

  committed
  e7a5289

• May's Review- employee_id is string in bob also should be not null

  RonAzar

  committed
  ad5e85d

• May's Review- should be one index on both columns, and the name should include the table name

  RonAzar

  committed
  934fdc7

• May's Review- Thank's for the review!

  RonAzar

  committed
  da4fb7d

• May's Review- Adding test migration

  RonAzar

  committed
  ea4d4ec

• Not finshed yet

  RonAzar

  committed
  f3be6db

• Create a function that retrieve all the pets by a given type (represented by enum) and return their name, dateOfArrival and company Id

  RonAzar

  committed
  128a9ac

Commits on Sep 11, 2024

• Create a function that retrieve all the owners from the owner table and return their name, employee Id and company Id.

  Show description for 20c9ff5
  RonAzar

  committed
  20c9ff5

• May review- should be bigint in hibob

  RonAzar

  committed
  b0b9a68

• May review- There is no point in putting an index on a primary key

  Show description for 788ccee
  RonAzar

  committed
  788ccee

• May review- maybe this file is causing you problems you can remove it

  RonAzar

  committed
  6a6ff04

• Complete

  RonAzar

  committed
  031a669

• unique index

  RonAzar

  committed
  28c5b5a

• Not finshed yet

  RonAzar

  committed
  48a1047

• Create a function that retrieve all the pets by a given type (represented by enum) and return their name, dateOfArrival and company Id

  RonAzar

  committed
  4d0de72

• Create a function that retrieve all the owners from the owner table and return their name, employee Id and company Id.

  Show description for f22849c
  RonAzar

  committed
  f22849c

• Complete class exercise

  RonAzar

  committed
  f12c49c

• Complete class exercise

  RonAzar

  committed
  ada0129

• Merge remote-tracking branch 'origin/RonAzar/JOOQ-Omer' into RonAzar/JOOQ-Omer

  Show description for 48175af
  RonAzar

  committed
  48175af

• Complete class exercise

  RonAzar

  committed
  ac1d80c

• Remove unique index from Pets

  RonAzar

  committed
  e22c14e

Commits on Sep 12, 2024

• change unique index to index

  RonAzar

  committed
  398c88d

• Change my enum class TYPE name to a more unique and representing name

  RonAzar

  committed
  e7b747a

• remove index from pets table

  RonAzar

  committed
  f178931

• Adding owner_id col to pets table

  RonAzar

  committed
  f0f5bf1

• Fixing my SQL-Migration PR

  RonAzar

  committed
  d47c154

• Lets move it to a different pr--> moved to my -Migration PR

  RonAzar

  committed
  8654173

• employeeId should be string field

  RonAzar

  committed
  246d4e1

• id is fine instead if ownerId

  RonAzar

  committed
  b12d769

• Improve my tests-

  Show description for eac350f
  RonAzar

  committed
  eac350f

• When we fetch data from the database we fetch by companyId(this is what we do in bob for security reasons). You need to get the companyId as parameter and fetch by companyId.

  RonAzar

  committed
  460a6cd

• Updated code

  RonAzar

  committed
  24683cf

• Add an API that will receive a pet id and owner id

  Show description for a773cab
  RonAzar

  committed
  a773cab

• Last commit Jooq

  RonAzar

  committed
  10bee44

• Merging from SQL branch

  RonAzar

  committed
  466a396

• Nothing

  RonAzar

  committed
  39492a6

• Adding data class with all pet info- PetAllInfo

  RonAzar

  committed
  69f5657

• Adding ownerId to PetTable

  RonAzar

  committed
  715fd62

Commits on Sep 13, 2024

• Complete JOOQ CRUD methods and corresponding tests.

  Show description for 9a5f1ed
  RonAzar

  committed
  9a5f1ed

• Switched to RIGHT JOIN to retrieve all owners, including those without pets. This ensures focus on owner data regardless of pet linkage.

  RonAzar

  committed
  ffa9dae

• Removed Pet redundant data class, keeping the one with complete pet information for better clarity and consistency.

  RonAzar

  committed
  958720c

• Spelling mistakes fixed

  RonAzar

  committed
  1b718e7

Commits on Sep 14, 2024

• Removed getPetByDetails function and added getPetById function. The insertNewPet function now returns the pet's serial ID, allowing for easier testing. Additionally, updatePetOwnerId now re…

  Show description for 78c459c
  RonAzar

  committed
  78c459c

• Fixed after last review

  Show description for ea30d4e
  RonAzar

  committed
  ea30d4e

• Little changes

  RonAzar

  committed
  bc31c81

• Removed companyId from test GetAllPets function() TEST

  RonAzar

  committed
  2efd2c3

• Adding 2 new Controllers

  RonAzar

  committed
  362ce75

• pets path

  RonAzar

  committed
  dcfa484

• add some comments

  RonAzar

  committed
  a20441a

• Adding new POST API Method to Owner

  RonAzar

  committed
  9b029fd

• Adding new POST API Method to get allOwners by company id

  RonAzar

  committed
  e4c38ef

Commits on Sep 15, 2024

• Complete Owner Resource

  RonAzar

  committed
  a5c287a

• Complete Pet Resource

  RonAzar

  committed
  e27f06e

• I should use a left join because it ensures that the query returns the pet even if it doesn’t have an associated owner, which matches your goal of always retrieving the pet’s information.

  RonAzar

  committed
  76f0fad

• Update JOOQ Version

  RonAzar

  committed
  04954a4

• Use resource now

  RonAzar

  committed
  dbe23e8

• Added new Resources

  RonAzar

  committed
  98392d9

• Completed implementation of Pet and Owner services and resources. Added API calls for all methods.

  RonAzar

  committed
  453168d

• Added few more tests to PetDao

  RonAzar

  committed
  a440455

• Added few more tests to PetDao & OwnerDao

  RonAzar

  committed
  bf6bf33

• Removed ?: -1 adding !!

  RonAzar

  committed
  a77d66f

Commits on Sep 16, 2024

• Removed A

  RonAzar

  committed
  69e03c1

• Removed All resources files and services

  RonAzar

  committed
  de6c08b

• Add All resources files and services

  RonAzar

  committed
  0cd64ef

• remove all changes from this migration it should be like this

  Show description for 4a5fab2
  RonAzar

  committed
  4a5fab2

• remove all changes from this migration it should be like this

  Show description for 675d9dc
  RonAzar

  committed
  675d9dc

• Merge remote-tracking branch 'origin/sql/lesson01' into sql/lesson01

  RonAzar

  committed
  dc1b5aa

• Merge branch 'master' into sql/lesson01

  RonAzar

  committed
  df3c502

• Merge branch 'sql/lesson01' into RonAzar/JOOQ-Omer

  RonAzar

  committed
  fc8059b

• Merge branch 'sql/lesson01' into RonAzar/JOOQ-Omer

  RonAzar

  committed
  71f844a

• Merge remote-tracking branch 'origin/RonAzar/JOOQ-Omer' into RonAzar/JOOQ-Omer

  RonAzar

  committed
  56262d6

• Merge branch 'RonAzar/JOOQ-Omer' into RonAzar/Resource-And-Service-Omer

  RonAzar

  committed
  e26b9f3

• Add 2 migrations vaccine and vaccineToPet (#25)

  Show description for 8ec4cdc
  RonAzar

  authored
  8ec4cdc

• Change my pet service and resource

  RonAzar

  committed
  c521e6b

• Add 2 migrations vaccine and vaccineToPet (#25)

  Show description for 4a9c9ec
  RonAzar

  committed
  4a9c9ec

• Merge remote-tracking branch 'origin/sql/lesson01' into sql/lesson01

  RonAzar

  committed
  cd8b3a5

• Start pet service tests

  RonAzar

  committed
  982967f

• Complete all service tests

  RonAzar

  committed
  c241403

• Merge branch 'RonAzar/JOOQ-Omer' into RonAzar/Resource-And-Service-Omer

  RonAzar

  committed
  7de5b6a

• Merge Jooq into current Branch

  RonAzar

  committed
  c85eef4

Commits on Sep 17, 2024

• Merge branch 'master' into RonAzar/Resource-And-Service-Omer

  RonAzar

  committed
  85975e0

• fixed test in github

  RonAzar

  committed
  88a0f29

• Fixed Pet resource and service

  RonAzar

  committed
  6d9e3cb

• Fixed Owner & Pet resource and service

  RonAzar

  committed
  dc19be2

• Fixed Owner & Pet resource and service

  RonAzar

  committed
  4014908

• Merge branch 'master' into RonAzar/Resource-And-Service-Omer

  RonAzar

  committed
  b29382d

• Change data classes to reasonable names

  RonAzar

  committed
  83c8009

• Update package path

  RonAzar

  committed
  f32692a

Commits on Sep 18, 2024

• Fixed - Why 2 twice companyId in path?

  RonAzar

  committed
  e2aec31

• Fixed - Response.ok is enough

  RonAzar

  committed
  1fcf6f7

• Fixed - No need to specify the next param

  RonAzar

  committed
  05e52f3

• Fixed - You already have @produces in the begging of the class, this is redundant.

  Show description for a0ff880
  RonAzar

  committed
  a0ff880

• Fixed - Why 2 twice companyId in path?

  RonAzar

  committed
  3554473

• Remove unnecessary '/'

  RonAzar

  committed
  0af0add

• Fixed- What we will do with this String?

  Show description for 35ff3ce
  RonAzar

  committed
  35ff3ce

• Fixed - enter between functions

  RonAzar

  committed
  2af9298

• Fixed - tests

  RonAzar

  committed
  3368774