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

🐛 fix: fix issue with private org membership on repos #34

Merged
merged 1 commit into from
Jul 12, 2024
Merged

🐛 fix: fix issue with private org membership on repos #34

merged 1 commit into from
Jul 12, 2024

Conversation

megasanjay
Copy link
Member

No description provided.

@fairdataihub-bot fairdataihub-bot
Copy link

Thank you for submitting this pull request! We appreciate your contribution to the project. Before we can merge it, we need to review the changes you've made to ensure they align with our code standards and meet the requirements of the project. We'll get back to you as soon as we can with feedback. Thanks again!

@sourcery-ai Sourcery AI
Copy link

sourcery-ai bot commented Jul 12, 2024 •
edited
Loading

🧙 Sourcery has finished reviewing your pull request!

Tips
  • Trigger a new Sourcery review by commenting @sourcery-ai review on the pull request.
  • Continue your discussion with Sourcery by replying directly to review comments.
  • You can change your review settings at any time by accessing your dashboard:
    • Enable or disable the Sourcery-generated pull request summary or reviewer's guide;
    • Change the review language;
  • You can always contact us if you have any questions or feedback.

@megasanjay megasanjay merged commit f0be58c into staging Jul 12, 2024
@fairdataihub-bot fairdataihub-bot
Copy link

Thanks for closing this pull request! If you have any further questions, please feel free to open a new issue. We are always happy to help!

sourcery-ai[bot]
sourcery-ai bot reviewed Jul 12, 2024
View reviewed changes
Copy link

@sourcery-ai sourcery-ai bot left a comment

Choose a reason for hiding this comment

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

We've reviewed this pull request using the Sourcery rules engine. If you would also like our AI-powered code review then let us know.

@what-the-diff What The Diff
Copy link

what-the-diff bot commented Jul 12, 2024

PR Summary

  • Enhancement in User Verification
    The update introduces a new feature that verifies if a user belongs to a specific organization. This is accomplished by interacting with the GitHub API. In case a user isn't part of the mentioned organization, an error notice will be sent. This enhancement serves to increase the security and ensure appropriate user access to specific organization resources in our application.

@megasanjay megasanjay deleted the private-orgs branch July 12, 2024 23:54
slugb0t added a commit that referenced this pull request Jul 13, 2024
@slugb0t @megasanjay
feat: ✨ bot updates + UI updates (#35)
cf6f1f9 
* wip: 🚧 auth

* wip: 🚧 auth

* wip: 🚧 implementing authentication

* wip 🚧 sending user to client

* 🐛 fix: update probot to esm (#11)

* 🐛 fix: update env var name

* 🐛 fix: update folder name

* 🐛 fix: update probot to esm

* ✨ feat: add basic license get page

---------

Co-authored-by: Dorian Portillo <wheresdorian@gmail.com>

* refactor: ♻️ update pnpm-lock file

* wip: 🚧 ensure validated session returns user/session

* refactor: ♻️ update pnpm lock file

* fix: 🐛 mongodb adapter returns user

* ✨ feat: add license ui (#12)

* 🐛 fix: update env var name

* 🐛 fix: update folder name

* 🐛 fix: update probot to esm

* ✨ feat: add basic license get page

* 🚧 wip: add base ui

* ✨ feat: add license ui

---------

Co-authored-by: Dorian Portillo <wheresdorian@gmail.com>

* feat: ✨ profile page + fix on existing user sessions

* 🚧 fix: auth (#13)

* 🚧 wip: protected page

* 🚧 fix: auth

* ✨ feat: add base pages and auth (#14)

* 🚧 wip: add mongoose

* fix: 🐛 mongoose adapter/callback

* refactor: ♻️ use mongoose on client end

* chore: 🔨 remove unused code

* 🚧 wip: test permissions

* 🚧 wip: test permissions

* 🚧 wip: test permissions

* 🚧 wip: test permissions

* 🚧 wip: test permissions

* ✨ feat: add error sections

* 🧑‍💻 chore: remove console.logs

---------

Co-authored-by: slugb0t <wheresdorian@gmail.com>

* ✨ feat: update layout

* 🚨 chore: fix the linters

* ✨ feat: add an error component

* 🐛 fix: use built in error component instead

* 🐛 fix: editor list styling

* 🐛 fix: add explanation text

* 🐛 fix: route to home on login

* ✨ feat: add support for license creation

* 🚨 fix: linter fixes

* 🔨 chore: update ping

* feat: ✨ issue redesign (#16) (#15)

* dev: 👷 yml file for creating probot app

* wip: 🚧 dashboard issue

* dev: 👷 nodemon for probot

* refactor: ♻️ provide context for renderissue fn + nodemon file watch

* wip: 🚧 provide localhost list depending on production

* feat: ✨ provide link to PR for dashboard

* feat: ✨ update dashboard on push

* wip: 🚧 abstracting code

* wip: 🚧 abstract functions of probot

* feat: ✨ dashboard workflow for LICENSE

* wip: 🚧 clean license UI

* feat: ✨ help text for license UI

* feat: ✨ renderer file has documentation + db update

* feat: ✨ documentation for tools

* feat: ✨ documentation for probot event listeners

* refactor: ♻️ file name instead of just name on UI/add license page

* wip: 🚧 styling for license UI

Co-authored-by: Dorian Portillo <wheresdorian@gmail.com>

* refactor: ♻️ update middleware

* 🚚 wip: test path

* 🚚 wip: test path

* 🚚 wip: test path

* 🚚 wip: test path

* 🚚 wip: test path

* 🚚 wip: test path

* 🚚 wip: test path

* refactor: ♻️ downgrade probot

* refactor: ♻️ update api imports

* refactor: ♻️ explicitly call github app credentials

* dev: 📝 update gitignore

* feat: ✨ run probot programatically

* feat: ✨ run probot v12 on a server

* feat: ✨ metadata UI + probot workflow (#23)

* chore: 🔨 remove unnecessary files

* ✨ feat: add preliminary support for codeMetadata ui (#19)

* 🛂 fix: update auth roles

* 🚧 wip: codemetada schema

* 🚧 wip: codemetada schema

* 🚧 wip: add base form

* 🚧 wip: add cards to sub forms

* 🚧 wip: basic form

* 🚧 wip: on authors

* 🚚 wip: add authors

* 🚧 wip: add contributors

* 🚧 wip: add contributors

* ✨ feat: add support for codeMetadata ui

* feat: ✨ probot server + metadata workflow  (#18)

* dev: 📝 add scripts for dev running probot

* dev: 📝 update pnpm-lock file

* feat: ✨ store owner and new time fields for License collection

* feat: ✨ update dev script to use nodemon

* wip: 🚧 repo metadata update

* fix: 🐛 push and create pr for license request

* bug: :fix: pass repo object to render issues

* fix: 🐛 es6 import for humanparser

* wip: 🚧 creating repo metadata db entry

* feat: ✨ function documentation

* feat: ✨ update repo name if it changes

* fix: 🐛 pass correct parameters for data fetch

* fix: 🐛 extra field needed for authors metadata

* feat: ✨ update db if repo name changes

* feat: ✨ update on codemeta template function

* fix: 🐛 parameters for file checking

* docs: 📝 documentation on license functions

* strictly pass repo name rather than entire object

* chore: 🔨 updates

---------

Co-authored-by: Sanjay Soundarajan <sanjay.16@live.com>

* 🐛 fix: draft saves

* 🐛 fix: draft saves

* refactor: ♻️ prettier refactoring

* refactor: ♻️ use correct path for codemeta link in issue dashboard

* fix: 🐛 use correct value for author names

* feat: ✨ contributors added to metadata object

* refactor: ♻️ update gather authors fn for contributors

* chore: 🔨 todo notes

* refactor: ♻️ empty contributors + provide correct issue tracker link + update gather authors fn

* fix: 🐛 installation collection fix

* chore: 🔨 updates and comments

* refactor: ♻️ revert gather authors fn

* fix: 🐛 save value for select pickers with one selection

* fix: 🐛 store form in metadata value

* fix: 🐛 show value if one avail on initial load (select picker)

* fix: 🐛 probot's codemeta values modified to suit UI

* fix: 🐛 correct value name for author/contrib roles

* refactor: ♻️ change timestamps to unixmilliseconds

* ✨ feat: add support for saving codemetadata in repository (#20)

* fix: 🐛 use correct comparison with reponame on db

* refactor: ♻️ use unix millisecond in metadata

* refactor: ♻️ verify repo name abstracted and patched

* fix: 🐛 correct value for operating sys, runtime plat for code metadta

* fix: 🐛 proper badge color for codemetada found

* feat: ✨ improved layout on metadata UI

* ✨ feat: add dashboards (#22)

* ✨ feat: add repo dashboard page

* ✨ feat: add github acount dashboard

* style: 🎨 UI design for code metadata page

* refactor: ♻️ shorten help text on issue dashboard for license

* refactor: ♻️ update help text on license UI

* feat: ✨ UI help text added for codemetata

---------

Co-authored-by: Sanjay Soundarajan <sanjay.16@live.com>

* refactor: ♻️ remove validate button from codemeta section

* Staging (#24)

* test

* changes

* remove new lines in private key

* push

* push

* pls

---------

Co-authored-by: slugb0t <wheresdorian@gmail.com>

* 👷 ci: use node 20 in nixpacks

* fix: 🐛 replace nanoid

* fix: 🐛 remove port

* refactor: ♻️ determine master branch within context

* ✨ feat: add support for login redirects

* refactor: ♻️ Bot feedback (#25)

* fix: 🐛 update title for dashboard issue

* refactor: ♻️ update no license template

* refactor: ♻️ update success license text

* refactor: ♻️ update no metadata found template

* feat: ✨ Display link to repo on license editor

* 🐛 fix: update ui and fix access token (#26)

* 🐛 fix: update ui and fix access token

* 🐛 fix: add read env for port

* 🐛 fix: remove duplicate decleration

* refactor: ♻️ update on badge buttons

* refactor: ♻️ add url to codemeta section

* refactor: ♻️ About section updated

* refactor: ♻️ strictly open links in new tab

* refactor: ♻️ remove redirect

* feat: ✨ analytics collection for db

* refactor: ♻️ save license content + id upon push to repo

* feat: ✨ save license content and id upon push to repo

* style: 🎨 rounded and shadow to dashboard cards

* refactor: ♻️ update on the github dashboard

* fix: 🐛 grammar fix on title dashboard

* refactor: ♻️ update to the code meta ui

* refactor: ♻️ hide editorial section of codemeta:

* refactor: ♻️ update on license UI

* refactor: ♻️ updates

* refactor: ♻️ timestamps formatted to human readable format

* 👷 ci: add support for webhook based deployments (#27)

* 💚 ci: add pnpm to the ci env

* 💚 ci: remove unused item

* ✨ feat: add beta tag to dashboard

* ✨ feat: add a contact form

* feat: ✨ Dashboard UI Changes + License and Codemeta.json extraction (#28)

* feat: ✨ collapsible cards integrated for repo dashboard

* refactor: ♻️ remove pre data and final touches for repo dashboard

* style: 🎨 theme for project + help text for metadata subsection

* style: 🎨 collapsiblecard changes

* 📌 update pnpm lock file

* fix: 🐛 input correct name in codemeta.json file

* wip: 🚧 import codemeta.json file if exists in repo

* fix: 🐛 correct values for codemeta.json extraction

* wip: 🚧 grab license content from existing license

* style: 🎨 update dropdown icons for large layout form

* refactor: ♻️ adjustments for extracting codemeta.json info

* refactor: ♻️ changes applied to codemeta.json extraction

* refactor: ♻️ authors/contributors extraction changes

* fix: 🐛 contributor extraction workflow updated

* feat: ✨ detect and extract codemeta.json from repo

* fix: 🐛 correct key values for contrib + authors

* refactor: ♻️ update for licenseId regex

* fix: 🐛 changes to author + contrib extraction

* fix: 🐛 contributors/authors fix

* refactor: ♻️ file cleanup

* feat: ✨ extra codemeta.json upon detection

* dev: 👷 update pnpm-lock file

* feat: ✨ remove repo from installation collection upon uninstallation

* feat: ✨ event listener for alternate method of uninstallation

* fix: 🐛 metadata import with empty values

* fix: 🐛 buffer license content with base64

* feat: ✨ help links added to metadata and license page

* feat: ✨ coming soon zenodo card

* refactor: ♻️ subheader for zenodo card

* 💄 style: update ui

* feat: ✨ theme change + general UI updates (#29)

* wip: 🚧 background change

* refactor: ♻️ update bg

* refactor: ♻️ improvements on svg background blur

* feat: ✨ healthcheck added to bot

* refactor: ♻️ update healthcheck endpoint

* feat: ✨ nixpacks config file

* refactor: ♻️ move toml file

* refactor: ♻️ add packagemananger in package.json

* refactor: ♻️ move toml

* refactor: ♻️ testing dockerfile update

* refactor: ♻️ update toml

* 👷 ci: disable secondary deployment

* refactor: ♻️ specify node version in package.json

* refactor: ♻️ delte files

* 👷 ci: disable secondary deployment

* refactor: ♻️ hide bg image until finalized

* refactor: ♻️ node version

* feat: ✨ need help links direct to documentation

* fix: 🐛 nav bar links

* Update README.md

* refactor: ♻️ spelling normalization for codefair

* refactor: dashboard issue text updates

* 💄 style: font changes

* 🐛 fix: am comments

* feat: ✨ check if repo is empty before workflow begins

* ✨ feat: add modals for confirmation of PRs (#33)

* feat: ✨ issue dashboard when repo is empty

* refactor: ♻️ update intro message of issue dashboard

* fix: 🐛 verify if license exists in codemeta extraction

* 🐛 fix: fix issue with private org membership on repos (#34)

* refactor: ♻️ standardize containers in codefair page

---------

Co-authored-by: Sanjay Soundarajan <sanjay.16@live.com>
Co-authored-by: Sanjay Soundarajan <ssoundarajan@calmi2.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sourcery-ai sourcery-ai[bot] sourcery-ai[bot] left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@megasanjay