Skip to content

Jonker/add link text#75

Closed
bryanjonker-illinois wants to merge 2 commits into
mainfrom
jonker/AddLinkText
Closed

Jonker/add link text#75
bryanjonker-illinois wants to merge 2 commits into
mainfrom
jonker/AddLinkText

Conversation

@bryanjonker-illinois

Copy link
Copy Markdown
Contributor

This pull request adds new API endpoints for loading Notes, FAQs, and Events via HTTP, allowing these entities to be created or updated through the API with proper authentication and logging. It also refactors existing controller classes for consistency, improves dependency injection setup, and streamlines variable declarations for readability.

New API Endpoints and Business Logic:

  • Added Load endpoints to Notes, Faqs, and Events controllers, enabling creation or update of these entities through the API. These endpoints require an API key, handle draft logic, and log submissions using LogHelper. (ResourceInformationV2.Function/Notes.cs [1] ResourceInformationV2.Function/Faqs.cs [2] ResourceInformationV2.Function/Events.cs [3]

Dependency Injection Improvements:

  • Registered new setter services (NoteSetter, FaqSetter, EventSetter, PublicationSetter) in the dependency injection container to support the new endpoints. (ResourceInformationV2.Function/Program.cs ResourceInformationV2.Function/Program.csR48-R56)

Constructor and Field Updates:

  • Updated constructors in Notes, Faqs, and Events to receive new dependencies (ApiHelper, LogHelper, and corresponding setter classes) via dependency injection. (ResourceInformationV2.Function/Notes.cs [1] ResourceInformationV2.Function/Faqs.cs [2] ResourceInformationV2.Function/Events.cs [3]

Code Style and Readability:

  • Refactored variable declarations in Notes and Faqs controllers to use var for consistency and readability. (ResourceInformationV2.Function/Notes.cs [1] [2]; ResourceInformationV2.Function/Faqs.cs [3] [4]

Minor Logging Consistency:

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