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 Api Handler #147

Merged
merged 8 commits into from
Feb 1, 2023
Merged

Create Api Handler #147

merged 8 commits into from
Feb 1, 2023

Conversation

Builditluc
Copy link
Owner

@Builditluc Builditluc commented Feb 1, 2023 •
edited
Loading

Linked Issues #143

Changes made:

  • feat: add mediawiki api struct with search working
  • feat: implement article_from_title for api handler
  • refactor: use map() for retrieving the article sections
  • feat: implement article_from_id for api handler
  • feat: implement search_at_offset for api handler
  • refactor: remove Search::origin and make result structs from api handler public
  • style: clean up the search offset fetching in search_from_json
  • refactor: create helper function for creating a query

@Builditluc Builditluc added state: approved This issue or pull request was approved and can be worked on type: refactor Changes to the style and structure of the codebase or project labels Feb 1, 2023
@Builditluc Builditluc self-assigned this Feb 1, 2023
@Builditluc Builditluc changed the base branch from main to 143-rework-backend February 1, 2023 16:12
@Builditluc Builditluc merged commit a1e2d44 into 143-rework-backend Feb 1, 2023
@Builditluc Builditluc deleted the 143-create-api-handler branch February 1, 2023 16:45
@Builditluc Builditluc mentioned this pull request Feb 2, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@Builditluc Builditluc

Labels
state: approved This issue or pull request was approved and can be worked on type: refactor Changes to the style and structure of the codebase or project
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@Builditluc