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

Create modules and organize code #44

Open
mindtheme opened this issue May 31, 2022 · 0 comments
Open

Create modules and organize code #44

mindtheme opened this issue May 31, 2022 · 0 comments
Assignees
@RumiAust @raghunandanarava
Labels
Est. Size 3 Real Size 3
Projects
OpenID Connect Doctor Backlogs

Comments

@mindtheme
Copy link
Collaborator

mindtheme commented May 31, 2022 •
edited

User Story

  1. As a software developer
  2. I want the code to be well organized
  3. So that the code can be easily extended

Acceptance Criteria

  • create user action pipeline
  • create modules to move code to
  • move code into modules

Definition of Done

  • Code has been tested
  • Optional Logging has been added
  • No errors (syntax, unit tests, etc.)
  • Peer reviewed by at least one additional person
  • Acceptance criteria met
  • The code was merged into the main branch via pull request
@mindtheme mindtheme created this issue from a note in OpenID Connect Doctor Backlogs (Product backlog) May 31, 2022
@mindtheme mindtheme moved this from Product backlog to Sprint backlog in OpenID Connect Doctor Backlogs Jun 1, 2022
@raghunandanarava raghunandanarava moved this from Sprint backlog to In progress in OpenID Connect Doctor Backlogs Jun 4, 2022
RumiAust added a commit that referenced this issue Jun 5, 2022
@RumiAust
discovery module - controller class code refactored #44
e15efb9
RumiAust added a commit that referenced this issue Jun 5, 2022
@RumiAust
unnecessary code removed from app controlled #44
34a90ad
raghunandanarava added a commit that referenced this issue Jun 5, 2022
@raghunandanarava
Migrated access token code to token module #44
2c277a2
raghunandanarava added a commit that referenced this issue Jun 6, 2022
@raghunandanarava
Added test cases for service #44
ad61e5f 
Signed-off-by: raghunandanarava <raghunandan.arava@fau.de>
raghunandanarava added a commit that referenced this issue Jun 6, 2022
@raghunandanarava
Changed the exception message for a check in requestToken method #44
66bbc4e
RumiAust added a commit that referenced this issue Jun 6, 2022
@RumiAust
discover service test case implemented #44
cc9f5f6
RumiAust added a commit that referenced this issue Jun 6, 2022
@RumiAust
login page added #44
2924e8a
RumiAust added a commit that referenced this issue Jun 6, 2022
@RumiAust
login page email frontend attributes added and linked with discovery …
ef3fcb7 
…page #44
RumiAust added a commit that referenced this issue Jun 6, 2022
@RumiAust
linked to login post method #44
a4a8c38
raghunandanarava added a commit that referenced this issue Jun 6, 2022
@raghunandanarava
Created flows module and defined a flow until getting an access token #…
32f8882 
…44
raghunandanarava added a commit that referenced this issue Jun 6, 2022
@raghunandanarava
Update dotenv file with a new variable ISSUER_STRING #44
6c5ef64
raghunandanarava added a commit that referenced this issue Jun 7, 2022
@raghunandanarava
Changed some return types #44
dc0e706
raghunandanarava added a commit that referenced this issue Jun 7, 2022
@raghunandanarava
Modified a few things in the client credentials workflow #44
93ad106
raghunandanarava pushed a commit that referenced this issue Jun 7, 2022
@RumiAust @raghunandanarava
discovery module - controller class code refactored #44
a2cce07
raghunandanarava pushed a commit that referenced this issue Jun 7, 2022
@RumiAust @raghunandanarava
unnecessary code removed from app controlled #44
10f8b1b
raghunandanarava added a commit that referenced this issue Jun 7, 2022
@raghunandanarava
Migrated access token code to token module #44
eefd5ff
raghunandanarava added a commit that referenced this issue Jun 7, 2022
@raghunandanarava
Added test cases for service #44
2aff4bb 
Signed-off-by: raghunandanarava <raghunandan.arava@fau.de>
raghunandanarava added a commit that referenced this issue Jun 7, 2022
@raghunandanarava
Changed the exception message for a check in requestToken method #44
403d7b6
raghunandanarava pushed a commit that referenced this issue Jun 7, 2022
@RumiAust @raghunandanarava
discover service test case implemented #44
27844a3
raghunandanarava pushed a commit that referenced this issue Jun 7, 2022
@RumiAust @raghunandanarava
login page added #44
35359dc
raghunandanarava pushed a commit that referenced this issue Jun 7, 2022
@RumiAust @raghunandanarava
login page email frontend attributes added and linked with discovery …
a87e728 
…page #44
raghunandanarava pushed a commit that referenced this issue Jun 7, 2022
@RumiAust @raghunandanarava
linked to login post method #44
28ffc6a
raghunandanarava added a commit that referenced this issue Jun 7, 2022
@raghunandanarava
Created flows module and defined a flow until getting an access token #…
3665696 
…44
raghunandanarava added a commit that referenced this issue Jun 7, 2022
@raghunandanarava
Update dotenv file with a new variable ISSUER_STRING #44
28c8cff
raghunandanarava added a commit that referenced this issue Jun 7, 2022
@raghunandanarava
Changed some return types #44
42012c7
raghunandanarava added a commit that referenced this issue Jun 7, 2022
@raghunandanarava
Modified a few things in the client credentials workflow #44
1e2b88a
raghunandanarava added a commit that referenced this issue Jun 7, 2022
@raghunandanarava
Included decode token #44
fa58daf
raghunandanarava added a commit that referenced this issue Jun 7, 2022
@raghunandanarava
Fixed test cases #44
8a3a5e7
FlinkbaumFAU added a commit that referenced this issue Jun 7, 2022
@FlinkbaumFAU
Merge pull request #52 from amosproj/44-create-modules-and-organize-code
f6b45e2 
44 create modules and organize code #44
@raghunandanarava raghunandanarava moved this from In progress to Awaiting review in OpenID Connect Doctor Backlogs Jun 8, 2022
@mindtheme mindtheme moved this from Awaiting review to Feature archive in OpenID Connect Doctor Backlogs Jun 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@RumiAust RumiAust

@raghunandanarava raghunandanarava

Labels
Est. Size 3 Real Size 3
Projects
OpenID Connect Doctor Backlogs
Feature archive
Milestone
No milestone
Development

No branches or pull requests
3 participants
@RumiAust @raghunandanarava @mindtheme