Skip to content
This repository was archived by the owner on Apr 24, 2024. It is now read-only.

Conversation

@hatchla
Copy link
Contributor

@hatchla hatchla commented Apr 21, 2024

Basics

  • The PR is rebased with current master
  • I added a line to changelog.md
  • Details of what I changed are in the commit messages
  • References to issues, e.g. close #X, are in the commit messages and changelog
  • The buildserver is happy

Checklist

  • I fully described what my PR does in the documentation
  • I fixed all affected documentation
  • I fixed the introduction tour
  • I wrote migrations in a way that they are compatible with already present data
  • I fixed all affected decisions
  • I added automated tests or a manual test protocol
  • I added code comments, logging, and assertions as appropriate
  • I translated all strings visible to the user
  • I mentioned every code or binary not directly written or done by me in reuse syntax
  • I created left-over issues for things that are still to be done
  • Code is conforming to our Architecture
  • Code is conforming to our Guidelines
  • Code is consistent to our Design Decisions
  • Exceptions to any guidelines are documented

First Time Checklist

Review

  • I've tested the code via issue description
  • I've tested the code via requirements
  • I've tested the code with concurrency (several browsers on the same map)
  • I've read through the code
  • I've read through the documentation
  • I've checked conformity to guidelines
  • I've checked conformity to requirements
  • I've checked that the requirements are automatically tested

@hatchla hatchla self-assigned this Apr 21, 2024
@hatchla hatchla linked an issue Apr 21, 2024 that may be closed by this pull request
2 tasks
@hatchla hatchla marked this pull request as ready for review April 21, 2024 19:34
@hatchla
Copy link
Contributor Author

hatchla commented Apr 21, 2024

Please state if you think something should be added or is unclear. Especially the part Implement Layer Logic might need some extenion or more details maybe.

@hatchla hatchla requested a review from badnames April 21, 2024 19:37
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.

Create tutorial for creating new layers in frontend

2 participants