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.

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

Hotfix: Spelling and Formatting in "Naming Conventions" #5

Merged

Conversation

TomChapple
Copy link

This hotfix introduces some minor fixes to the "Naming Conventions" section where there are minor inconsistencies or missing information.

Of large note are the addition of headings in order to avoid bleeding examples into the "Managing Dates" section. This section may need to be reviewed to meet appropriate standards.

The example is inconsistent with its employee ID. That is, the employee is initially referred to with ID "E13454" and is then retrieved with a GET at `/project-name/v1/employees/E13455`. The latter now ends with `E13454`.
Although there was a heading for "Bad URL examples", there was no equivalent for "Good URL examples" leading it to bleed into the "Date Field Naming Conventions" section. In addition, it also bleeds into the more generic "Managing Dates" section, so a higher-level heading was created called "Examples".
To keep the "Bad URL examples" section consistent with "Good URL examples", HTTP methods were added that were related to the operation they were applying.
Copy link

@jordanwalsh23 jordanwalsh23 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good pickups. As these are minor formatting and consistency changes I'm happy to approve these for merge.

Thanks for the contribution.

@colugo colugo merged commit 44aada3 into apigovau:master Dec 18, 2019
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.

3 participants