Skip to content
This repository was archived by the owner on Nov 10, 2025. It is now read-only.

Add deprecation notice#838

Merged
lfdebrux merged 1 commit intomainfrom
ldeb-add-deprecation-notice
Oct 2, 2025
Merged

Add deprecation notice#838
lfdebrux merged 1 commit intomainfrom
ldeb-add-deprecation-notice

Conversation

@lfdebrux
Copy link
Copy Markdown
Contributor

@lfdebrux lfdebrux commented Oct 1, 2025

What problem does this pull request solve?

Trello card: https://trello.com/c/BxY1aMOZ/2496-update-team-documentation-now-we-no-longer-use-forms-api

forms-api is no longer required in any part of our service and will be removed soon. Add a note to the repo to inform people of this.

Things to consider when reviewing

  • Ensure that you consider the wider context.
  • Does it work when run on your machine?
  • Is it clear what the code is doing?
  • Do the commit messages explain why the changes were made?
  • Are there all the unit tests needed?
  • Has all relevant documentation been updated?

forms-api is no longer required in any part of our service and will be
removed soon. Add a note to the repo to inform people of this.
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Oct 1, 2025

@lfdebrux lfdebrux merged commit d811d70 into main Oct 2, 2025
5 checks passed
@lfdebrux lfdebrux deleted the ldeb-add-deprecation-notice branch October 2, 2025 11:10
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants