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.

Sign up for GitHub

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

Jump to bottom

🤔 [ISSUE] - Update updateOne logic to centralize logs #576

Closed
yassinedorbozgithub opened this issue Jan 16, 2025 · 0 comments · Fixed by #579
Closed

🤔 [ISSUE] - Update updateOne logic to centralize logs #576

yassinedorbozgithub opened this issue Jan 16, 2025 · 0 comments · Fixed by #579
Assignees
@yassinedorbozgithub
Labels
enhancement New feature or request

Comments

@yassinedorbozgithub
Copy link
Collaborator

Description

We can notice that we have a repetitive code related to the updateOne methods (repository, service).

Reproduction steps

1. Go to '...'
2. Click on '....'
3. Scroll down to '....'
4. See error

Logs



Browsers


No response


OS


No response

      		
    

      
  





        



            

              
            

        

      


      
    








      

    



      

  
            

    

      
    

    


      

          @yassinedorbozgithub
yassinedorbozgithub




          added
  the 

  enhancement

  New feature or request
 label


      Jan 16, 2025

    

  



    

    

      
    

    


      

          @yassinedorbozgithub
yassinedorbozgithub




              self-assigned this


      Jan 16, 2025

    

  



  

  

    
  


  

    


      @yassinedorbozgithub
yassinedorbozgithub



        linked a pull request
        Jan 16, 2025
        that will
        close
        this issue





  

    
      update(api): centralize updateOne error handling logic
      #579
  



  

      
   Merged

  





  
  

      
        
      
      

  
    
    
  


    2 tasks
  





  




  

  

    
  


  

      

    @yassinedorbozgithub
yassinedorbozgithub



    mentioned this issue
    
      Jan 16, 2025
    





  


  

      
   Merged

  





  
  

      
        
      
      

  
    
    
  


    2 tasks
  





  




    

    

      
    

    


      

          @yassinedorbozgithub
yassinedorbozgithub




        changed the title
🤔 [ISSUE] - Refactor updateOne logic to centralize logs

🤔 [ISSUE] - Update updateOne logic to centralize logs


      Jan 17, 2025

    

  



    

    











  
  

    

      

  





  




  
  

              

  
    Sign up for free
    to join this conversation on GitHub.
    Already have an account?
    Sign in to comment


  



  






  
                  




    

  


      
  

    Assignees
  



        

      

        

  
      @yassinedorbozgithub
  
    yassinedorbozgithub
    

      








      


  


  

    Labels
  



    

      

    enhancement

  New feature or request








      

  

    

        

    Projects
  


        




    None yet




  



      


  

    
  

    Milestone
  


      No milestone





    
      
          


  

    
      

        
    

    Development
  




          
  
Successfully merging a pull request may close this issue.





    


  
          
      update(api): centralize updateOne error handling logic

    
      Hexastack/Hexabot
    





    
  




      

    

  
      

  

    

      1 participant
    

    

        
          @yassinedorbozgithub