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

add new company - Accor #1657

Merged
merged 2 commits into from
Oct 8, 2023
Merged

add new company - Accor #1657

merged 2 commits into from
Oct 8, 2023

Conversation

dhruvchandak30
Copy link
Contributor

@dhruvchandak30 dhruvchandak30 commented Oct 1, 2023 •
edited
Loading

This is a modified version of, and should adhere to, the Contributing Guidelines.

  • This PR contains housekeeping only (URL edits, copy changes etc) - if this is checked, delete other lines that don't apply
  • You are an employee of the company mentioned and confirm all included details are correct.
  • You know your alphabet - company is listed in alphabetical order in the README.md file.
  • The company directly hires employees. ** No bootcamps / freelance sites / etc**
  • The company hires remote employees, or positions are available to remote workers and are clearly illustrated as such.
  • A company profile is included - This is required for new additions.
  • An entry has been added or edited within the readme.md file.
  • Remote status has details regarding how the culture includes remote employees, how the company integrated remote workers, etc
  • Region details the geographic regions in which this company employees can reside. For more details see the instructions in the example company profile.
  • How to apply details the best approach for new applications, page on site where open position are listed, and any other help available for job hunters

@dhruvchandak30 dhruvchandak30 mentioned this pull request Oct 1, 2023
Open
@Ki-er Ki-er added the addition label Oct 1, 2023
@dhruvchandak30
Copy link
Contributor Author

@KieranRobson Can you please add hacktoberfest and hacktoberfest-accepted labels.
It would be a great help for me.

dougaitken
dougaitken requested changes Oct 1, 2023
View reviewed changes
company-profiles/accor.md Outdated

## Region

France, Issy-les-Moulineaux
Copy link
Member

Choose a reason for hiding this comment

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

Region is where the company hires - the section above lists 110 countries so I'd recommend using worldwide if they hire remotely in those countries

@dougaitken
Copy link
Member

@dhruvchandak30 The repository is labelled hacktoberfest so you'll get the participation.

Can you check my review please?

@dhruvchandak30
Changed Region to Worldwide
69c60c5 
Signed-off-by: Dhruv Chandak <[email protected]>
@dhruvchandak30
Copy link
Contributor Author

dhruvchandak30 commented Oct 1, 2023 •
edited
Loading

@dhruvchandak30 The repository is labelled hacktoberfest so you'll get the participation.

Can you check my review please?
@dougaitken
Yes, Done with the changes

@dhruvchandak30
Copy link
Contributor Author

dhruvchandak30 commented Oct 2, 2023 •
edited
Loading

@KieranRobson @dougaitken Can you please merge this PR, as I have pushed from my main branch, I am not able to make more PR (I am but it is also showing these two commits).

@Ki-er Ki-er added the rtm Ready to merge label Oct 2, 2023
@dougaitken
Copy link
Member

as I have pushed from my main branch, I am not able to make more PR (I am but it is also showing these two commits).

best to use branches for new features rather than main 😉

@dougaitken dougaitken merged commit 409b5c3 into remoteintech:main Oct 8, 2023
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dougaitken dougaitken dougaitken approved these changes

Assignees
No one assigned
Labels
addition rtm Ready to merge
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@dhruvchandak30 @dougaitken @Ki-er