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

Api services #21

Open
wants to merge 34 commits into
base: main
Choose a base branch
from
Open

Api services #21

wants to merge 34 commits into from

Conversation

sam-baraka
Copy link
Contributor

@sam-baraka sam-baraka commented Nov 2, 2022

Scope

Please make sure to read the mobile development guide provided.
and check that you understand and have followed it as best as possible Explain what your feature
does in a short paragraph.
please check the below boxes

  • I have followed the coding conventions
  • I have added/updated necessary tests
  • I have tested the changes added on a physical device

Closes/Fixes Issues

Declare any issues by typing fixes #1 or closes #1 for example so that the automation can kick
in when this is merged

Other testing QA Notes

What have you tested specifically and what possible impacts/areas there are that may need retesting
by others.

Please add a screenshot (if necessary)

Contribution

For every push that you make , make user to update the build number

 version: 0.5.0+1
 
 +1 is the build number

For every bug fixed or updated on a feature that you make , make user to update the patch number

 version: 0.5.0+1
 
 0.5.patchNumber+1

For every new feature and functionality completion , make user to update the minor version number

 version: 0.5.0+1
 
 0.minorVersionNumber.0+1

Before you push your changes to make sure you update the changelog and indicate changes made.
A new changelog entry will be added on patches / minor version build / releases or major release .
Change log will follow this formate.

## Added = > for new features.
## Changed for  =>  changes in existing functionality.
## Deprecated => for soon-to-be removed features.
## Removed  => for now removed features.
## Fixed => for any bug fixes.
## Security = > in case of vulnerabilities.

Copy link
Contributor

@KendiJ KendiJ left a comment

Choose a reason for hiding this comment

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

Seen

Copy link
Collaborator

@mulieriq mulieriq left a comment

Choose a reason for hiding this comment

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

Resolve merge conflict

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